This test against MAX_CAS_ID is not needed: C standard _guarantees_
that ++ for unsigned type will wraparound to zero automagically. And
this condition won't be ever met anyway, so the check is a bit of a
waste (unless you have some reasons not to use full 64 bit range that
I'm failing to see, of course).
@@ -697,11 +698,15 @@ static void complete_nread(conn *c) {
...
+ else if(ret == 3)
+ out_string(c, "NOT FOUND");
Should be "NOT_FOUND" (with underscore). This typo was present in the
original code, and was accidentally copied here.
Both fixed in r636, plus explicitly initialized the cas_id to zero for
portability. :)
-Dormando
