To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=96055
Issue #|96055
Summary|idlc: ambiguous && || sequence
Component|udk
Version|DEV300m35
Platform|All
URL|
OS/Version|Linux
Status|NEW
Status whiteboard|
Keywords|
Resolution|
Issue type|PATCH
Priority|P3
Subcomponent|code
Assigned to|kr
Reported by|cmc
------- Additional comments from [EMAIL PROTECTED] Mon Nov 10 14:59:03 +0000
2008 -------
in idlc/source/astscope.cxx there is...
if ( (NT_service == m_nodeType) &&
((pDecl->getNodeType() == NT_interface_member)
&& (pDeclaration->getNodeType() == NT_interface)) ||
((pDecl->getNodeType() == NT_service_member)
&& (pDeclaration->getNodeType() == NT_service)))
taking the three bracketed terms this is effectively
if ( X && Y || Z)
and so binds as
if ( (X && Y) || Z )
but looking at the names of the terms it looks a little ambiguous. Might the
intent actually be...
if ( X && (Y || Z) )
i.e.
if (
(NT_service == m_nodeType) &&
(
((pDecl->getNodeType() == NT_interface_member)
&& (pDeclaration->getNodeType() == NT_interface))
||
((pDecl->getNodeType() == NT_service_member)
&& (pDeclaration->getNodeType() == NT_service))
)
)
or is the intent truly...
if (
(
(NT_service == m_nodeType) &&
((pDecl->getNodeType() == NT_interface_member)
&& (pDeclaration->getNodeType() == NT_interface))
) ||
((pDecl->getNodeType() == NT_service_member)
&& (pDeclaration->getNodeType() == NT_service))
)
---------------------------------------------------------------------
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]
