#5664 closed change (fixed)
[emscripten] Change `assert` macro to accept a single parameter.
| Reported by: | sergz | Assignee: | hfiguiere |
|---|---|---|---|
| Priority: | P4 | Milestone: | |
| Module: | Core | Keywords: | |
| Cc: | trev | Blocked By: | |
| Blocking: | #4122 | Platform: | Unknown / Cross platform |
| Ready: | yes | Confidential: | no |
| Tester: | Unknown | Verified working: | no |
| Review URL(s): | |||
Description
Background
assert is an expected standard macro, therefore when one includes standard headers after our definition of the macro the code cannot be compiled.
What to change
- change it to accept only a single parameter
- consider adding some assert2 accepting a description as the second argument or some analog.
Change History (5)
comment:1 Changed 3 years ago by hfiguiere
- Owner set to hfiguiere
- Review URL(s) modified (diff)
comment:2 Changed 3 years ago by hfiguiere
- Status changed from new to reviewing
comment:3 Changed 3 years ago by abpbot
comment:4 Changed 3 years ago by hfiguiere
- Resolution set to fixed
- Status changed from reviewing to closed
comment:5 Changed 3 years ago by trev
- Priority changed from P5 to P4
- Ready set
For reference, priority is usually set by module owner or peers. Also, P5 is used for issues that shouldn't be worked on.
Note: See
TracTickets for help on using
tickets.
A commit referencing this issue has landed:
Issue 5664 - assert now accept one parameter.