To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=96834
User cloph changed the following:
What |Old value |New value
================================================================================
CC|'sb' |'cloph,sb'
--------------------------------------------------------------------------------
------- Additional comments from [email protected] Thu Mar 26 13:32:22 +0000
2009 -------
Sure that the patch is correct? To me it doesn't really make it less ambiguous.
if (long)n>nMin it will make the whole if-clause true but the parentheses assign
it to the preceding && statement which is kind of void, isn't it?
Before:
... || (nSign == 1 && n >= 0x8000000 || (long)n > nMin)
After (just to make the compiler happy, but not really make the statement
clear):
... || ((nSign == 1 && n >= 0x8000000) || (long)n > nMin)
both are the same as
(long)n > nMin || ... || (nSign == 1 && n >= 0x8000000)
Is this really intended? The initial grouping suggest something different...
---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
