[xml-issues] [Issue 100502] oox: gcc finds operations || ambigous

2009-09-22 Thread dr
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=100502


User dr changed the following:

What|Old value |New value

  Status|VERIFIED  |CLOSED





--- Additional comments from d...@openoffice.org Tue Sep 22 08:09:47 + 
2009 ---
closed

-
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: issues-unsubscr...@xml.openoffice.org
For additional commands, e-mail: issues-h...@xml.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[xml-issues] [Issue 100502] oox: gcc finds operations || ambigous

2009-05-15 Thread dr
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=100502


User dr changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from d...@openoffice.org Fri May 15 16:17:01 + 
2009 ---
verified in CWS

-
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: issues-unsubscr...@xml.openoffice.org
For additional commands, e-mail: issues-h...@xml.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[xml-issues] [Issue 100502] oox: gcc finds operations || ambigous

2009-03-24 Thread mav
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=100502


User mav changed the following:

What|Old value |New value

 Summary|oox: ambiguous  ||  |oox: gcc finds operations 
|  | || ambigous





--- Additional comments from m...@openoffice.org Tue Mar 24 09:40:35 + 
2009 ---
Changing the title

-
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: issues-unsubscr...@xml.openoffice.org
For additional commands, e-mail: issues-h...@xml.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[xml-issues] [Issue 100502] oox: gcc finds operations || ambigous

2009-03-24 Thread dr
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=100502


User dr changed the following:

What|Old value |New value

  Status|REOPENED  |RESOLVED

  Resolution|  |FIXED





--- Additional comments from d...@openoffice.org Tue Mar 24 09:43:47 + 
2009 ---
changed implementation accordingly, thanks for your feedback.

-
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: issues-unsubscr...@xml.openoffice.org
For additional commands, e-mail: issues-h...@xml.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[xml-issues] [Issue 100502] oox: gcc finds operations || ambigous

2009-03-24 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=100502





--- Additional comments from c...@openoffice.org Tue Mar 24 09:51:43 + 
2009 ---
FWIW, while in this case the implementation was correct, in about 40 out of 45
previous cases the precedence rules were not what was intended so it would
appear to be a quite useful warning. Though as we're down to the small subset of
gray ones where it the code is neither obviously wrong or obvious right I guess
the ratio will change

-
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: issues-unsubscr...@xml.openoffice.org
For additional commands, e-mail: issues-h...@xml.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[xml-issues] [Issue 100502] oox: gcc finds operations || ambigous

2009-03-24 Thread mba
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=100502





--- Additional comments from m...@openoffice.org Tue Mar 24 13:21:33 + 
2009 ---
Mikhail, I also thought at first that these kinds of warnings are superfluous.
But I learned and saw that they revealed some bugs in other places, where
(different to your case) the code was not working as the sloppy developer
wanted. So it makes a lot of sense to set braces even if they are not necessary.
It also makes the code easier to read and understand, and that's a good thing 
also.

-
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: issues-unsubscr...@xml.openoffice.org
For additional commands, e-mail: issues-h...@xml.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[xml-issues] [Issue 100502] oox: gcc finds operations || ambigous

2009-03-24 Thread mav
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=100502





--- Additional comments from m...@openoffice.org Tue Mar 24 14:21:28 + 
2009 ---
If the warning has helped to find and fix around 40 bugs from 45 suspicious
cases, it indeed makes sense. 

I did not pretend to be error prove in case of logical expressions. It is
probably a matter of taste, for me additional brackets, especially brackets in
brackets, make in most cases big expressions less readable.

-
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: issues-unsubscr...@xml.openoffice.org
For additional commands, e-mail: issues-h...@xml.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org