[vtk-developers] Add pugixml to VTK

[Jonathan Borduas]

Last time I checked, ParaView was using an older version of PugiXML (1.4) while there are now at version 1.9. 

I think the benchmarks are considering the version 1.9. If you just port the current ParaView implementation of PugiXML to VTK, you might be disappointed.  

Best,

Jonathan

-----Original Message-----
From: vtk-developers <vtk-developers-bounces at public.kitware.com> On Behalf Of David Thompson
Sent: Friday, June 15, 2018 1:19 PM
To: Utkarsh Ayachit <utkarsh.ayachit at kitware.com>
Cc: vtk-developers at vtk.org
Subject: Re: [vtk-developers] Add pugixml to VTK

>> 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.
>> 
>> https://pugixml.org/benchmark.html
>> 
>> Are there any comments or concerns about moving pugixml from ParaView to VTK?

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...

	David
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html

Search the list archives at: http://markmail.org/search/?q=vtk-developers

Follow this link to subscribe/unsubscribe:
https://public.kitware.com/mailman/listinfo/vtk-developers