Review: Needs Fixing

Interesting! However, XMLString::transcode() returns a memory buffer that the 
caller must free by calling XMLString::release(), so I believe this change 
introduces a memory leak (unless InputSource will free them, which I don't 
think it does). I'm actually not 100% sure how to solve this - I guess 
IstreamInputSource could free them in its destructor, if it can tell that it 
needs to do so?
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to     : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

Reply via email to