bitlogik / uniblow

UNIversal BLOckchain Wallet
GNU General Public License v3.0
19 stars 12 forks source link

Remove assert #18

Closed bitlogik closed 1 year ago

bitlogik commented 1 year ago

All the assert keywords in the code base need to removed (except in tests). That will improve the user experience, by providing dedicated and friendly exception message. And also will unify the experience between running from source and the binaries. The binaries are compiled with the assert removed, so the assert doesn't act as check guard when running the binaries. For that reason, the assert keyword should not be present in a production software, hence need to be removed.