nsTextFrame.cpp has the following declaration: 70 #ifndef PR_ABS 71 #define PR_ABS(x) (x < 0 ? -x : x) 72 #endif and PR_ABS is later used as: if (found) { 2033 PRInt32 charWidth; 2034 acx->GetWidth(text[indx], charWidth); 2035 charWidth /= 2; 2036 2037 if (PR_ABS(PRInt32(aPoint.x) - origin.x) > textWidth+charWidth) { 2038 indx++; 2039 } 2040 } Seems like more parentheses are needed... 71 #define PR_ABS(x) ((x) < 0 ? -(x) : (x))

Ccing evaughan because of a similar definition in nsSplitterFrame.cpp. (This is just to raise awareness since the macro is not used there --yet!)

Checked in patch supplied by rbs@maths.uq.edu.au

Marking verified per last comments.