search

decred / tinydecred

Python tools for Decred
ISC License
27 stars 14 forks source link

multi: use unblobCheck all over and remove AssertionErrors #132

Closed teknico closed 4 years ago

teknico commented 4 years ago

Move unblob_check to the top package and use it all over the place. Remove all AssertionErrors except for one.

teknico commented 4 years ago

Is the idea from here to start subclassing DecredError to get more refined error handling?

That too, as much as useful, and that all errors raised by the library are DecredError subclasses, so that callers can rely on that.

teknico commented 4 years ago

Can we move it to encode instead?

Also, can we change it to unblobCheck, for consistency.

Makes sense, doing both.