search

TaylanUB / scheme-bytestructures

Structured access to bytevector contents.
GNU General Public License v3.0
36 stars 6 forks source link

-helper is a bad name #5

Closed johnwcowan closed 8 years ago

johnwcowan commented 8 years ago

Using -helper chooses identifiers based on purpose rather than function. Better to give the triple of bytevector, descriptor, and offset a name such as "location", and then use "bytestructure-location" instead of "bytestructure-ref-helper".

TaylanUB commented 8 years ago

I'm having difficulties coming up with a good name. Maybe bytestructure-unwrap?

TaylanUB commented 8 years ago

I just pushed a commit that changes all occurrences of "ref-helper" to either "unwrapper" or "unwrap" depending on context. I like the name and am closing the ticket for now, but further suggestions still welcome. (Especially since it's past 1 AM and I might be making bad decisions.)