Posted by toniw on October 6, 2009 at 7:16pm
Jump to:
| Project: | Nodequeue |
| Version: | 7.x-2.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
If I set the max length of a queue to n it will never be allowed to grow larger than n-1.
So if I set the max length to 5 then adding the fifth node will already bump the first node from the queue, limiting the queue to a max of 4 elements.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| nodequeue-maxlength-offbyone.patch | 530 bytes | Ignored: Check issue status. | None | None |
Comments
#1
You're right -- I changed this function around recently and missed the -1.
Thanks for the attention to -dev and the patch!
Committed.
#2
Automatically closed -- issue fixed for 2 weeks with no activity.