You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Follow-up to discussion in PR #71 , as well as issue #68 .
There's already some use of assert() throughout the codebase, but there could be more - for instance more argument validation (such as calling diag_l0_send with len == 0), etc.
To have those always-enabled (regardless of NDEBUG), it could be a better idea to use a new macro or wrapper func with our own assert implementation.
The text was updated successfully, but these errors were encountered:
Follow-up to discussion in PR #71 , as well as issue #68 .
There's already some use of
assert()
throughout the codebase, but there could be more - for instance more argument validation (such as calling diag_l0_send with len == 0), etc.To have those always-enabled (regardless of NDEBUG), it could be a better idea to use a new macro or wrapper func with our own assert implementation.
The text was updated successfully, but these errors were encountered: