PersistentVector can't handle reverse iteration starting from last element

GlenKPeterson / Paguro

Generic, Null-safe, Immutable Collections and Functional Transformations for the JVM

Other

312 stars 24 forks source link

PersistentVector can't handle reverse iteration starting from last element #8

Closed pniederw closed 8 years ago

pniederw commented 8 years ago

If index == size(), then array = null, and previous() will throw NPE.

Also, listIterator() doesn't do any bounds checks (unlike UnmodList.listIterator()).

GlenKPeterson commented 8 years ago

Thank you! This is fixed in 1.0.3 which has been deployed to Maven. Also as a result of this, I greatly improved the tests for listIterator and found and fixed bugs in RangeOfInt and UnmodList. You can see details and your credit in the change log (and in the diff).