> 
> 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)
> 

I wouldn't add the "(e.g. because ....)" part. After all, a temporary node 
becomes not available by being deleted. So, the phrase does not really add any 
additional info.

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

I am not sure, but these 2 errors are being used just for internal sanity 
checks. If so, they should be removed from the diagnostics list (use 
ZORBA_ASSERT() instead).

> 
> 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);
> 

I made this change because I changed theNodeToReferencesMap to be a hashmap 
(instead of std::map), and in doing so, I renamed some of the HashMap methods 
to better match the std::map methods.
-- 
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