zeux / pugixml

Light-weight, simple and fast XML parser for C++ with XPath support
http://pugixml.org/
MIT License
4.01k stars 728 forks source link

resolved all microsoft native recommened warnings #597

Closed matthewhelyar closed 11 months ago

matthewhelyar commented 11 months ago

Resolved all the Microsoft Native Recommended warnings. Compiled using C++14 on MSVC, but should be backwards compatible further than that. Wasn't sure what version to target.

matthewhelyar commented 11 months ago

I can't get the default initialization of arrays to work in the AppVeyor tests on Visual Studio version 14 or below as the warnings about default initialization of T = char and and T = wchar_t are treated as errors in those tests. (Talk about irony...) All other CI pass. You are welcome to all the other fixes, but the uninitialized variables are the source of most of the warnings I was trying to resolve. I've therefore closed the PR.

6fbb5c0 passed the most tests