>I don't think that the Head/Tail concept applies well here because there's no
>indication when you insert whether you're inserting at the head or tail.  You
>could rename Insert to Push, and RemoveFirst to Pop as the function is closer
>to a stack than a list.  Or you could name it
>IndexListRemoveEntryThatWasInsertedLast. :)

I changed it to IndexListRemoveHead().

There is a doubly-linked list that's being maintained, and the item being
removed is at the head of that list.  It's not necessarily the item that was
inserted last, but whatever item ended up at the head.  Since items can be
removed in the middle of the list, it doesn't match the behavior of a stack.

Now, if you want to argue that IndexListInsert() would be better named
IndexListInsertHead()...

- Sean

_______________________________________________
ofw mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw

Reply via email to