Review: Needs Fixing

<p>Please see the <a href="../../html/data_lifecycle.html">data lifecycle       
                   
 documentation</a> about details on management and manipulation of 
collections.</p>
=>
<p>Please see the <a href="../../html/data_lifecycle.html">data lifecycle       
                   
 documentation</a> about details on storing nodes in collections or as 
documents.</p>


Computes an immutable and opaque node reference 
=>
Returns an immutable and opaque node reference...

The function returns the empty sequence if the node
that is referenced was deleted.
=>
The function returns the empty sequence if the node
that is referenced does not exist (e.g. because it was deleted or the temporary 
node is not available anymore)

zerr::ZAPI0028
=>
zerr:ZAPI0028

Can any of the functions return ZAPI0029? If so, it's not documented. Same for 
ZAPI0030.


fix indentation in NodeByReferenceIterator (two spaces)
Also, add a space before and after the assignment

Fix comment in FnGenerateIdIterator => remove e.g. z0.1.1.c.50

Why is hasReference still needed?

What is the following change in node_items.cpp for?
513     -    theTypesMap = new NodeTypeMap(source->theTypesMap->object_count(), 
false);
514     +    theTypesMap = new NodeTypeMap(source->theTypesMap->size(), false);

What is the CopyMode's theCopyReference member for? Isn't it always false?
-- 
https://code.launchpad.net/~zorba-coders/zorba/identifiers/+merge/78383
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