Decouple XML reading from the CLR PAL
delmyers opened this issue · comments
Nagler needs to read XML to run its scripts. Currently, it uses msxml to read xml, but we have code inside InstrumentationEngine that can read xml using msxml or libxml. Unfortunately, this code is coupled with the PAL to do encoding/decoding and to pass strings around as BSTR types. This is not necessary. We should decouple this xml reading code from bstrs, and use iconv to do the encoding/decoding if necessary on Linux. The encoding/decoding will not be needed on Windows.