Code review comment for lp:~zorba-coders/zorba/abstract-collection-set

Revision history for this message
Matthias Brantner (matthias-brantner) wrote :

- Did you forget to add file simple_collection_set.h
- I would keep the CollectionSet in the simplestore namespace because it's a component defined by the simplestore and not the store api (i.e. a file in store/api)

review: Needs Fixing

« Back to merge proposal