[
https://issues.apache.org/jira/browse/FORTRESS-24?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12582830#action_12582830
]
Jim Apple commented on FORTRESS-24:
-----------------------------------
This will also cause a NullPointerException for a null className.
> Useless length check in
> fortress/meta/src/java/org/apache/avalon/fortress/tools/Component.java
> ----------------------------------------------------------------------------------------------
>
> Key: FORTRESS-24
> URL: https://issues.apache.org/jira/browse/FORTRESS-24
> Project: Excalibur Fortress
> Issue Type: Bug
> Components: Implementation
> Affects Versions: 1.1
> Reporter: Jim Apple
> Priority: Minor
> Original Estimate: 0.08h
> Remaining Estimate: 0.08h
>
> http://www.google.com/codesearch?hl=en&q=+%5Cw%7B9%7D%5C+%5C!%5C%3D%5C+null%5C+%5C%7C%5C%7C%5C+%5Cw%7B9%7D%5C.%5Cw*%5C(+show:Hj1pNaTZhMg:DqZL3QbYD5E:qjugdXajwK4&sa=N&cd=1&ct=rc&cs_p=http://apache.edgescape.com/excalibur/excalibur-fortress/source/avalon-fortress-tools-1.1-src.zip&cs_f=avalon-fortress-tools-1.1/src/java/org/apache/avalon/fortress/tools/Component.java#first
> line 368:
> if ( className != null || className.length() > 0 )
> the || should probably be &&
--
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
