-
Notifications
You must be signed in to change notification settings - Fork 9
Replace assert()
calls with if
/else
/raise <Exception>()
#28
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
Comments
Can I take this? |
@alimustafashah Thanks for your interest, for sure thank you very much, that would be really appreciated. thanks :) |
Can you assign the task to me? |
@alimustafashah if this is your first time contributing to CP Thanks :) |
Thanks for sharing! |
#28 Replaced assert() calls with if/else/raise <Exception>()
Solved by #30 |
Per a conversation in the FXAS21002C library on issue #21, checking for conditions and raising an exception is preferable over calling
assert()
. This is preferable for a number of reasons, including memory usage.The text was updated successfully, but these errors were encountered: