> In archive_module.cpp:
> 1. Please don't leave commented-out code. Just delete it; it's in the version
> history if we ever need to refer back to it. For example, remove these lines:
> +    //Item lNameItem = theFactory->createString(lName);
> +    //lMemberName = theFactory->createString("name");
> 2. As discussed, if possible, create single zorba::Items for all the JSON
> object key strings, eg. "name", "size", etc. Possibly create those in the
> ArchiveModule constructor. Use those constants when creating JSON objects in
> EntriesIterator::next(), as well as comparing input values in
> ArchiveEntry::setValues() (and anywhere else it happens to be appropriate).
> Also don't forget to assign a bug to yourself to create a JSON schema for this
> module, and to go back and update the module and test cases accordingly.

Your team Zorba Coders is subscribed to branch lp:zorba/archive-module.

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