[prev in list] [next in list] [prev in thread] [next in thread] 

List:       xmlbeans-dev
Subject:    [jira] Assigned: (XMLBEANS-348) 2 Dubious assignments, probably
From:       "Jacob Danner (JIRA)" <xmlbeans-dev () xml ! apache ! org>
Date:       2007-11-08 5:14:50
Message-ID: 15856808.1194498890854.JavaMail.jira () brutus
[Download RAW message or body]


     [ https://issues.apache.org/jira/browse/XMLBEANS-348?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

Jacob Danner reassigned XMLBEANS-348:
-------------------------------------

    Assignee: Jacob Danner

> 2 Dubious assignments, probably should be equality tests
> --------------------------------------------------------
> 
> Key: XMLBEANS-348
> URL: https://issues.apache.org/jira/browse/XMLBEANS-348
> Project: XMLBeans
> Issue Type: Bug
> Affects Versions:  Version 2.3
> Environment: n/a
> Reporter: Charles Butterfield
> Assignee: Jacob Danner
> 
> While peeking at the source code to report bug XMLBEANS-347, I (or rather the \
> compiler) ran across two dubious assignments that were probably meant to be \
> equality tests as shown below: 1) org.apache.xmlbeans.impl.common.XPath: line 717:
> if ((_lastDeepDot = deepDot))
> NOTE: if this really is meant to be an assignment, then the following conditional \
> "_lastDeepDot = true;" is clearly redundant, which helped me conclude this is a \
> bug, rather than an evil combined assignment and test. 2) \
> org.apache.xmlbeans.impl.schema.SchemaTypeSystemImpl: line 756: assert \
> assertEnabled = true; Hope you find this useful,
> Regards
> -- Charlie

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@xmlbeans.apache.org
For additional commands, e-mail: dev-help@xmlbeans.apache.org


[prev in list] [next in list] [prev in thread] [next in thread]