alecthomas / mph

Minimal Perfect Hashing for Go
BSD 3-Clause "New" or "Revised" License
172 stars 23 forks source link

slicereader: Use a local variable instead of a struct member #15

Closed Jille closed 7 months ago

Jille commented 7 months ago

It took me a second to understand that b.start isn't actually used to persist anything. Using a local variable for this seems simpler.

Also exposing it as a single variable makes my next commit a tiny bit easier to understand.