search

rust-bakery / nom

Rust parser combinator framework
MIT License
9.18k stars 792 forks source link

[nit] Replace hard-coded bit count with constant. #1684

Closed ttsugriy closed 2 months ago

ttsugriy commented 10 months ago

This makes it clear what 8 stands for.

Geal commented 8 months ago

I'm not sure it makes things easier to read. If you work with bits, you know why 8 is there