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
We have ASSERT(), but it isn't an assert, it's just a check-and-warn. And it's always included, there's no debug / release split.
This is confusing for devs, especially new devs, who may expect that ASSERTS a) only exist in debug builds b) when included, prevent later code from executing if the assert fails.
WARN() may be a more appropriate name. Note there is also a DryOS assert(), which is a function pointer so might not assert, but in practice always does. We also, optionally, replace this function pointer with our own (that does more verbose logging then transfers to DryOS original debug_assert().
The text was updated successfully, but these errors were encountered:
reticulatedpines
changed the title
ML ASSESRT() doesn't assert, rename
ML ASSERT() doesn't assert, rename
Jan 28, 2024
We have ASSERT(), but it isn't an assert, it's just a check-and-warn. And it's always included, there's no debug / release split.
This is confusing for devs, especially new devs, who may expect that ASSERTS a) only exist in debug builds b) when included, prevent later code from executing if the assert fails.
WARN() may be a more appropriate name. Note there is also a DryOS assert(), which is a function pointer so might not assert, but in practice always does. We also, optionally, replace this function pointer with our own (that does more verbose logging then transfers to DryOS original debug_assert().
The text was updated successfully, but these errors were encountered: