<div dir="ltr">David,<div>This is certainly an option. There are many readers that require external libraries.</div><div>Maybe this is the better choice. Moving pugixml would be a future step when/if other modules start using it.</div><div><br></div><div>Dan</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 15, 2018 at 1:19 PM David Thompson <<a href="mailto:david.thompson@kitware.com">david.thompson@kitware.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>> I am working on a CityGML reader for VTK. For this I need a fast XML parser. pugixml is already used by ParaView and seems to be much faster than available choices already present in VTK: libxml2 and expat. pugixml has also a very clean API.<br>
>> <br>
>> <a href="https://pugixml.org/benchmark.html" rel="noreferrer" target="_blank">https://pugixml.org/benchmark.html</a><br>
>> <br>
>> Are there any comments or concerns about moving pugixml from ParaView to VTK?<br>
<br>
I thought Ben Boeckel was working on a removing as many things from ThirdParty as possible (and making them parts of a superbuild). So perhaps the CityGML reader should live in a VTK module that requires an external pugi? I know pugi is small, but...<br>
<br>
        David</blockquote></div>