search

binast / binjs-ref

Reference implementation for the JavaScript Binary AST format
https://binast.github.io/binjs-ref/binjs/index.html
Other
431 stars 38 forks source link

Get rid of `Enrich::scopes` #410

Closed Yoric closed 5 years ago

Yoric commented 5 years ago

I introduced this option a while ago to make sure that a refactoring didn't affect the behavior of test_roundtrip. Turns out that this behavior was wrong all along and now that it has been fixed, the only reasonable value for scopes is true.

We should get rid of scopes everywhere.