Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error message not concise #251

Open
patrickwieth opened this issue Feb 11, 2019 · 1 comment
Open

Error message not concise #251

patrickwieth opened this issue Feb 11, 2019 · 1 comment

Comments

@patrickwieth
Copy link

So this is not a real bug, but an error message that could be more helpful:

https://github.com/tendermint/go-amino/blob/master/amino.go#L337

The error message fits well for the case in L339 but in L337 it should not inform that the prefix bytes are wrong but are missing. I thought I messed up registering the codec or something like this but actually the data was empty due to another bug.

@liamsi
Copy link
Contributor

liamsi commented Feb 11, 2019

Thanks @patrickwieth!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants