On Nov 27, 2007, at 3:26 PM, Dan Janowski wrote:
I see the merit in this kind of approach but it cannot conflict with the libxml work flow. I.e.: instead of XML::Document.parse(xml) => Document XML::Parser.parse(xml) => Document If you want to update the patch for the current code base, I am willing to apply and eval it.
I updated the original patch from Tobias to work with the current subversion trunk (220). I made the suggested modification above so it's XML::Parser.parse(xml) rather than XML::Document.parse -- though I do think XML::Document.parse is a little bit of a cleaner API.
I also found a bug with namespace assignments, if you assign a namespace to a node not associated with a document it segfaults:
doc = XML::Document.new
node = XML::Node.new('root')
node.namespace = "t:test"
I'm not sure what the best way to fix this is since I'm not the
familiar with the namespace code at this point.
Thanks, Paul
libxml-patched.tar.gz
Description: GNU Zip compressed data
On Nov 27, 2007, at 13:48, Paul Dlug wrote:I see patch #7758 hasn't been worked on or updated since submission (long ago): http://rubyforge.org/tracker/index.php? func=detail&aid=7758&group_id=494&atid=1973 This seems like a great idea and the new parse method solves eliminates the need for part of the patch I submitted (#15807). Is there any interest in getting this into the current library? I would be happy to modify the patch to bring it up to date with the current trunk version. This would certainly create a much more user friendly API than what currently exists. Thanks, Paul _______________________________________________ libxml-devel mailing list libxml-devel@rubyforge.org http://rubyforge.org/mailman/listinfo/libxml-devel_______________________________________________ libxml-devel mailing list libxml-devel@rubyforge.org http://rubyforge.org/mailman/listinfo/libxml-devel
_______________________________________________ libxml-devel mailing list libxml-devel@rubyforge.org http://rubyforge.org/mailman/listinfo/libxml-devel
