[jira] Commented: (FORTRESS-24) Useless length check in fortress/meta/src/java/org/apache/avalon/fortress/tools/Component.java

by Excalibur - Dev mailing list :: Rate this Message:

[ https://issues.apache.org/jira/browse/FORTRESS-24?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12570626#action_12570626 ]

Jim Apple commented on FORTRESS-24:
-----------------------------------

version 1.2 as well,

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:jypgqn3okUM:Ph0xIl6hzIE:SlX648yLaVA&sa=N&cd=2&ct=rc&cs_p=http://apache.edgescape.com/excalibur/excalibur-fortress/source/excalibur-fortress-meta-1.2-src.zip&cs_f=excalibur-fortress-meta-1.2/src/java/org/apache/avalon/fortress/tools/Component.java#first

line 411

> 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: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

« Return to Thread: [jira] Created: (FORTRESS-24) Useless length check in fortress/meta/src/java/org/apache/avalon/fortress/tools/Component.java

LightInTheBox - Buy quality products at wholesale price!