search

overengineering / rustassonne

0 stars 0 forks source link

Connections on Features should be a set not a vector #6

Closed nadinengland closed 7 years ago

nadinengland commented 7 years ago

After documenting Issue #5 it seemed sensible to change the connections: Vec<direction> to a Set, presumably a HashSet. This is because a Feature only every touches a direction once. Future expansions may bring rule changes (e.g. a road that loops back on itself in one tile?) but let's not pre-optimise for this.

StefanoChiodino commented 7 years ago

but let's not pre-optimise for this.

@nadinengland that sounds like under-engineering. You have been warned.