In NonblockingQueue::try_pop there is a local variable named "result". That choice of name seems to make the code and comments harder to read. A better name should be used; old_head has been suggested.