commit d29c7cf1aa069db722337e4a6c79b0a848a90465
author Carsten Ziegeler <cziegeler@apache.org> Wed Apr 08 12:59:13 2009 +0000
committer Carsten Ziegeler <cziegeler@apache.org> Wed Apr 08 12:59:13 2009 +0000
tree cd51e7e15c71e1241402399c0772569cc2acf902
parent 1993d108af0f6efdec8a69eb5a6e91c3022c430b

FELIX-1028 : Throw NPE instead of InvalidSyntaxException if expr is null.

git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@763227 13f79535-47bb-0310-9956-ffa450edef68

framework/src/main/java/org/apache/felix/framework/FilterImpl.java

diff --git a/framework/src/main/java/org/apache/felix/framework/FilterImpl.java b/framework/src/main/java/org/apache/felix/framework/FilterImpl.java
index 6086e80..4b7abfb 100644
--- a/framework/src/main/java/org/apache/felix/framework/FilterImpl.java
+++ b/framework/src/main/java/org/apache/felix/framework/FilterImpl.java
@@ -1,4 +1,4 @@
-/* 
+/*
  * Licensed to the Apache Software Foundation (ASF) under one
  * or more contributor license agreements.  See the NOTICE file
  * distributed with this work for additional information
@@ -20,8 +20,8 @@
 
 import java.io.CharArrayReader;
 import java.io.IOException;
-import java.util.*;
 import java.lang.ref.SoftReference;
+import java.util.*;
 
 import org.apache.felix.framework.util.StringMap;
 import org.apache.felix.framework.util.ldap.*;
@@ -56,7 +56,7 @@
         m_logger = logger;
         if (expr == null)
         {
-            throw new InvalidSyntaxException("Filter cannot be null", null);
+            throw new NullPointerException("Filter cannot be null");
         }
         Object[] program = null;
         synchronized (m_programCache)
@@ -127,7 +127,7 @@
         return toString().hashCode();
     }
 
-    private boolean match(Dictionary dict, ServiceReference ref, boolean caseSensitive) 
+    private boolean match(Dictionary dict, ServiceReference ref, boolean caseSensitive)
         throws IllegalArgumentException
     {
         SoftReference tupleRef = (SoftReference) m_cache.get();
@@ -135,36 +135,36 @@
         SimpleMapper mapper = null;
         Object[] tuple = null;
 
-        if (tupleRef != null) 
+        if (tupleRef != null)
         {
             tuple = (Object[]) tupleRef.get();
         }
 
-        if (tuple == null) 
+        if (tuple == null)
         {
             evaluator = new Evaluator(m_program);
             mapper = new SimpleMapper();
         }
-        else 
+        else
         {
             evaluator = (Evaluator) tuple[0];
             mapper = (SimpleMapper) tuple[1];
             m_cache.set(null);
         }
 
-        try 
+        try
         {
-            if (dict != null) 
+            if (dict != null)
             {
                 mapper.setSource(dict, caseSensitive);
             }
-            else 
+            else
             {
                 mapper.setSource(ref);
             }
 
             return evaluator.evaluate(mapper);
-        } 
+        }
         catch (AttributeNotFoundException ex)
         {
             log(Logger.LOG_DEBUG, "FilterImpl: Attribute not found.", ex);
@@ -173,18 +173,18 @@
         {
             log(Logger.LOG_ERROR, "FilterImpl: " + toString(), ex);
         }
-        finally 
+        finally
         {
-            if (dict != null) 
+            if (dict != null)
             {
                 mapper.setSource(null, caseSensitive);
             }
-            else 
+            else
             {
                 mapper.setSource(null);
             }
-            
-            if (tuple == null) 
+
+            if (tuple == null)
             {
                 m_cache.set(new SoftReference(new Object[] {evaluator, mapper}));
             }