Edge-case path is broken
cscott opened this issue · 1 comments
cscott commented
The fix for bug #3 (PR #4) contained some code in addArray that attempted to handle the case where the "highest two" array indexes were already in use. However, the corresponding code in containsArray seems to have been overlooked/forgotten. The containsArray function should also check to see if the highest two array indexes are in use, and if so undertake an exhaustive search for the elements added by addArray.
cscott commented
Never mind, that code is crazy like a fox. Never thought array_slice behaved that way. Cool, I guess?