← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/jsoniq into lp:zorba

 

Review: Needs Information

I've reviewed the serialization changes. One tiny issue: I think in jsoniq_emitter::emit_end(), you should call emit_end() on either theXMLEmitter or theJSONEmitter depending on theEmitterState. I know that right now, neither the XML nor the JSON emitters actually do anything in emit_end(), but if they ever need to do something in the future, this will be a bug.
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/92171
Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/jsoniq into lp:zorba.


References