Review: Needs Fixing

The commit looks good to me. However, could you please put an item int he 
ChangeLog that mentions the api extension?

Also, for consistency and performance, the results of the Unmarshaller calls in 
createComment/PiNode should be "zstring const&".
-- 
https://code.launchpad.net/~zorba-coders/zorba/ItemFactory/+merge/86125
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