From 739833c51230d2bdb7699906b7acaa6a8bb76044 Mon Sep 17 00:00:00 2001
From: Jon Bratseth <bratseth@yahoo-inc.com>
Date: Thu, 8 Jun 2017 11:11:19 +0200
Subject: Nonfunctional changes

---
 .../prelude/querytransform/PhraseMatcher.java      | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

(limited to 'container-search/src/main/java/com/yahoo/prelude/querytransform/PhraseMatcher.java')

diff --git a/container-search/src/main/java/com/yahoo/prelude/querytransform/PhraseMatcher.java b/container-search/src/main/java/com/yahoo/prelude/querytransform/PhraseMatcher.java
index 12a904a29e2..7c6d1285454 100644
--- a/container-search/src/main/java/com/yahoo/prelude/querytransform/PhraseMatcher.java
+++ b/container-search/src/main/java/com/yahoo/prelude/querytransform/PhraseMatcher.java
@@ -12,13 +12,13 @@ import java.util.NoSuchElementException;
 import static com.yahoo.language.LinguisticsCase.toLowerCase;
 
 /**
- * <p>Detects query phrases using an automaton. This class is thread safe.</p>
+ * Detects query phrases using an automaton. This class is thread safe.
  *
- * @author  bratseth
+ * @author bratseth
  */
 public class PhraseMatcher {
 
-    private FSA phraseFSA;
+    private FSA phraseFSA = null;
 
     private boolean matchPhraseItems=false;
 
@@ -31,7 +31,7 @@ public class PhraseMatcher {
     private boolean matchAll =false;
 
     /** For null subclass only */
-    PhraseMatcher() {
+    private PhraseMatcher() {
     }
 
     /**
@@ -69,6 +69,8 @@ public class PhraseMatcher {
         phraseFSA=phraseAutomatonFSA;
     }
 
+    public boolean isEmpty() { return phraseFSA == null; }
+    
     /**
      * Set whether to match words contained in phrase items as well.
      * Default is false - don't match words contained in phrase items
@@ -138,10 +140,10 @@ public class PhraseMatcher {
     }
 
     /** Find matches within a composite */
-    private void recursivelyMatchPhrases(Item item,MatchedPhrases phrases) {
-        if (item==null) return;
+    private void recursivelyMatchPhrases(Item item, MatchedPhrases phrases) {
+        if (item == null) return;
         if ( ! (item instanceof CompositeItem) ) return;
-        if ( !matchPhraseItems && item instanceof PhraseItem ) return;
+        if ( ! matchPhraseItems && item instanceof PhraseItem ) return;
 
         CompositeItem owner=(CompositeItem)item;
         int i=0;
@@ -547,12 +549,16 @@ public class PhraseMatcher {
 
     /** Returns a phrase matcher which (quickly) never matches anything */
     public static PhraseMatcher getNullMatcher() {
+
         return new PhraseMatcher() {
 
-                public List<Phrase> matchPhrases(Item item) {
+            @Override
+            public List<Phrase> matchPhrases(Item item) {
                     return null;
                 }
+
         };
+
     }
 
 }
-- 
cgit v1.2.3