search

indutny / macho

Mach-O parser for node.js
23 stars 10 forks source link

es6 #6

Open trufae opened 8 years ago

trufae commented 8 years ago

it would be nice to start using const and let at least, but there are other interesting es6 features that could be used here, users can still use old versions of the macho module if they are using an ancient js interpreter. but all browsers support that now and it's also supported by node.

indutny commented 8 years ago

I don't mind.

trufae commented 8 years ago

const bring some performance goodies and let allows to clarify the lifetime of variables and such, there are several other benefits in es6 , not just from the syntactic POV. this issue is probably more a discussion to see what other users think about this switch I guess :)

On 31 Jul 2016, at 20:15, Fedor Indutny notifications@github.com wrote:

I don't mind.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/indutny/macho/issues/6#issuecomment-236447101, or mute the thread https://github.com/notifications/unsubscribe-auth/AGIjG6xQmgPs2vBeWX7oMljpWtxXJiXKks5qbOY0gaJpZM4JYy9B.