zorba-coders team mailing list archive
-
zorba-coders team
-
Mailing list archive
-
Message #05971
Re: [Merge] lp:~28msec/zorba/get-descendant-node-by-ordpath into lp:zorba
Within the for loop:
for (i = 0; i < numAttrs; i++)
in the case of (pos == OrdPath::DESCENDANT) the function should return false, no?
Also, please use csize (instead of ulong) for storing the size of vectors and for iterating over vectors.
Also, in for loops, do ++i instead of i++. It's a very tiny optimization, but it doesn't cost anything so why let it pass....
--
https://code.launchpad.net/~28msec/zorba/get-descendant-node-by-ordpath/+merge/95902
Your team Zorba Coders is subscribed to branch lp:zorba.
References