Author: rickhall
Date: Fri Apr 13 19:54:42 2012
New Revision: 1325928

URL: http://svn.apache.org/viewvc?rev=1325928&view=rev
Log:
Fix bug in Util.isOptional().

Modified:
    felix/trunk/resolver/src/main/java/org/apache/felix/resolver/Util.java

Modified: felix/trunk/resolver/src/main/java/org/apache/felix/resolver/Util.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/resolver/src/main/java/org/apache/felix/resolver/Util.java?rev=1325928&r1=1325927&r2=1325928&view=diff
==============================================================================
--- felix/trunk/resolver/src/main/java/org/apache/felix/resolver/Util.java 
(original)
+++ felix/trunk/resolver/src/main/java/org/apache/felix/resolver/Util.java Fri 
Apr 13 19:54:42 2012
@@ -75,8 +75,7 @@ public class Util
     public static boolean isOptional(Requirement req)
     {
         String resolution = 
req.getDirectives().get(Namespace.REQUIREMENT_RESOLUTION_DIRECTIVE);
-        return ((resolution == null)
-            || resolution.equalsIgnoreCase(Namespace.RESOLUTION_OPTIONAL));
+        return Namespace.RESOLUTION_OPTIONAL.equalsIgnoreCase(resolution);
     }
 
     public static List<Requirement> getDynamicRequirements(List<Requirement> 
reqs)


Reply via email to