Review: Needs Fixing

- nice job
- comments of the functions in xqxq.xq are missing
- std::stringstream* lExcPlan = new std::stringstream(); 
  will leak if saveExecutionPlan returns false or raises an error. You should 
use an auto_ptr or something similar
- Chris do you have ideas for better/nicer function names?
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1123016/+merge/155082
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