Change storage of sketch details from HashSet to ArrayList

richsmith / sexytopo

The SexyTopo cave surveying app for Android

GNU General Public License v3.0

30 stars 13 forks source link

Change storage of sketch details from HashSet to ArrayList #149

Closed danielworkman closed 3 years ago

danielworkman commented 3 years ago

Changed the storage of all four sketch detail types from HashSet to ArrayList
This a) decreases iteration time from O(h/n), (where h is the capacity) to O(1) and b) retains the original ordering
Also removed an unused import

richsmith commented 3 years ago

Yep, seems sensible. Not sure if it'll make that much difference but all helps. I imagine this will make deleting things etc. slower but we can easily afford that.