From: Francesco M. <f18...@ya...> - 2007-01-03 10:30:16
|
Hi, cecilio ha scritto: > Hi Francesco, > Sorry for the delay. > >>> could you submit it as a patch? >>> It's much easier to apply in that form (just do a "cvs diff -bu2 > >sample.patch" in wxXml2\sample). > > Well I am very new to CVS and SVN so I dont know how to prepare the > patch. But in any case, I moved to SVN and currently I don't have CVS > installed, so I can not execute the suggested command. well actually I did forget that I moved wxXml2 to SVN so that it's in wxCode SVN and not in wxCode CVS... ;) > Therefore I am sending you the full modified source file. this is ok anyway. > I built the > sample and tested it. It works ok: external entities are detected and > replaced correctly. There was a bug in the code I sketched in my > previous e-mail, as it only worked ok if the xml file being processed > has only one external entity defined. If it has more than one > definition, I have discovered that after a node of type > wxXML_ENTITY_REF_NODE you will find, as children, the full list of > declared external entities. So I had to modify the code to choose the > proper one. The attached code is tested and works ok. > > Hope this is OK for you. Sure, thanks! I'll check it in asap. Francesco |