Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Remove support for non-minimal VarInts #140

Open
prestwich opened this issue Apr 3, 2020 · 1 comment
Open

Remove support for non-minimal VarInts #140

prestwich opened this issue Apr 3, 2020 · 1 comment
Labels
all-languages Needs work in all languages enhancement New feature or request

Comments

@prestwich
Copy link
Member

Turns out this was unnecessary the whole time 🎉

We should error on these instead of accept them.

https://twitter.com/_prestwich/status/1246159783490510848

@prestwich prestwich added enhancement New feature or request all-languages Needs work in all languages labels Apr 3, 2020
@prestwich
Copy link
Member Author

This is done for Solidity in #158
this is partially done for C in a local branch

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
all-languages Needs work in all languages enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant