Re: Abs() macro in sysutil.h is incorrect

[rflores]

>Actually there should be parens around the first "a" too:
>      #define Abs(a) (((a) >= 0) ? (a) : -(a))

Consider this now fixed.

Until you get headers with this change, I suggest adding

#undef Abs
#define Abs(a) (((a) >= 0) ? (a) : -(a))


>My rule of thumb is to always put parens around EVERY occurence of a
>macro argument in the expansion.

This is very good advice.


Thanks for the report.


-Roger Flores