bug 117479 ("RFE: Implement a PR_ABS() macro") introduced the PR_ABS() macro to NSPR headers. Unfortunately many people use things like: -- snip -- #undef PR_ABS #define PR_ABS (x>0)?x:-x -- snip -- which is completly wrong (missing brackets about the |x|-usage in the macro and the PR_* CPP namespace gets misused). I'll file a patch to kill the crap and do some other ABS()-macro-related cleanup...
Assignee: asa → Roland.Mainz
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla0.9.9
Comment on attachment 69568 [details] [diff] [review] Patch sr=sfraser
Attachment #69568 - Flags: superreview+
Patch checked in, marking bug as FIXED.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.