opr: Don't confuse isLast and isEnd
authorSimon Wilkinson <sxw@your-file-system.com>
Wed, 21 Nov 2012 16:46:29 +0000 (16:46 +0000)
committerDerrick Brashear <shadow@your-file-system.com>
Fri, 23 Nov 2012 14:18:31 +0000 (06:18 -0800)
commitb28c51d594ba3147bc1c30c34b974beb3f1fdfb3
tree59b11d37fa2211853bd6d53a643f0ed8e39d7aba
parentad4e634051e18fa5bd07016b6405e53e236c2f45
opr: Don't confuse isLast and isEnd

opr_queue_IsEnd's implementation was incorrect - it would return
true when the element was the last item in the list, not when it
was the end of the list (equal to the head record)

Correct the implementation of isEnd, and add an implementation for
isLast.

This fixes a bug in RX, wher we would never notice that the last
packet in the transmit queue was acknowledged, because the loop that
iterates over the queue uses isEnd to detect when its work is done.

Change-Id: I8966e05c479c18d025bb5cc4cf77514ce002be95
Reviewed-on: http://gerrit.openafs.org/8493
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
src/opr/queue.h
tests/opr/queues-t.c