search

no-context / moo

Optimised tokenizer/lexer generator! 🐄 Uses /y for performance. Moo.
BSD 3-Clause "New" or "Revised" License
830 stars 66 forks source link

Refactor to use `type` instead of `name` internally #28

Closed nathan closed 7 years ago

nathan commented 7 years ago

It's somewhat confusing that token types are called names until they are emitted, so we should probably refer to them everywhere as types.

Tim has a bunch of cool PRs in the works right now that touch that code, though, so let's wait until those are merged.

tjvr commented 7 years ago

We should do this right after #25.

tjvr commented 7 years ago

FWIW, the reason I didn't do this is it seemed nicer to call a rule/group by its "name" than by its "type".

nathan commented 7 years ago

That is a good point. I suppose you could call it tokenType?