[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-11-26 Thread sb
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379


User sb changed the following:

What|Old value |New value

  Status|VERIFIED  |CLOSED





--- Additional comments from [EMAIL PROTECTED] Wed Nov 26 08:36:43 + 
2008 ---
.

-
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]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

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


User cmc changed the following:

What|Old value |New value

OtherIssuesDependingOnTh|  |96084
  is|  |





-
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]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-11-07 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379


User cmc changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from [EMAIL PROTECTED] Fri Nov  7 16:15:38 + 
2008 ---
set to verified as its the agreed patch

-
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]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-11-05 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379


User cmc changed the following:

What|Old value |New value

 Assigned to|cmc   |sb





--- Additional comments from [EMAIL PROTECTED] Wed Nov  5 16:57:33 + 
2008 ---
cmc-sb: happy to verify this in cmcfixes50 ?

-
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]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-10-23 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379
 Issue #|95379
 Summary|tools: ambiguous operator precedence
   Component|utilities
 Version|DEV300m34
Platform|All
 URL|
  OS/Version|Linux
  Status|NEW
   Status whiteboard|
Keywords|
  Resolution|
  Issue type|PATCH
Priority|P3
Subcomponent|code
 Assigned to|cd
 Reported by|cmc





--- Additional comments from [EMAIL PROTECTED] Thu Oct 23 15:02:06 + 
2008 ---
So, my recent gcc has some extra warnings about dodgy looking implicit operator
precedence.

The following patch just enforces the current implicit operator precedence, i.e.
 before  before | before  before || so it changes nothing except to make
those warnings go away.

It all looks ok except maybe for source/fsys/urlobj.cxx where we have...
while (p != pEnd  INetMIME::isHexDigit(*p) || *p == ':' || *p == '.')
where a  b || c || d is effectively (((a  b) || c) || d) i.e.
while ((p != pEnd  INetMIME::isHexDigit(*p)) || *p == ':' || *p == '.')
is this what is truly intended or is it intended to be
while (p != pEnd  (INetMIME::isHexDigit(*p) || *p == ':' || *p == '.'))

-
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]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-10-23 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379


User cmc changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |57417)
patch, that assumes
|  | implicit precedence is co
|  |rrect






--- Additional comments from [EMAIL PROTECTED] Thu Oct 23 15:03:09 + 
2008 ---
Created an attachment (id=57417)
patch, that assumes implicit precedence is correct


-
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]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-10-23 Thread cd
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379


User cd changed the following:

What|Old value |New value

  Status|NEW   |STARTED





--- Additional comments from [EMAIL PROTECTED] Thu Oct 23 15:11:21 + 
2008 ---
cd-sb: Could you please check the patch from Caolan.

-
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]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-10-23 Thread sb
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379


User sb changed the following:

What|Old value |New value

 Assigned to|cd|cmc

  Status|STARTED   |NEW





--- Additional comments from [EMAIL PROTECTED] Thu Oct 23 15:27:13 + 
2008 ---
@cmc:  The fix for source/fsys/urlobj.cxx should read

  while (p != pEnd  (INetMIME::isHexDigit(*p) || *p == ':' || *p == '.'))

(i.e., the current code is wrong, and the extra warning helped find an error). 
Either integrate yourself, or pass back to me and I will add it to some 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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-10-23 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379


User cmc changed the following:

What|Old value |New value

  Status|NEW   |STARTED

Target milestone|---   |OOo 3.1





--- Additional comments from [EMAIL PROTECTED] Thu Oct 23 15:35:53 + 
2008 ---
will take for cmcfixes50

-
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]



[util-issues] [Issue 95379] tools: ambiguous operator precedence

2008-10-23 Thread cmc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=95379


User cmc changed the following:

What|Old value |New value

  Status|STARTED   |RESOLVED

  Resolution|  |FIXED





--- Additional comments from [EMAIL PROTECTED] Thu Oct 23 15:36:43 + 
2008 ---
done

-
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]