Merge pull request #24567 from vespa-engine/bratseth/cleanup-01

Add a test and rename a private method

1 files changed, 3 insertions, 3 deletions

diff --git a/container-search/src/main/java/com/yahoo/search/yql/YqlParser.java b/container-search/src/main/java/com/yahoo/search/yql/YqlParser.java
index 3a1927983d2..1203f69dd3c 100644
--- a/container-search/src/main/java/com/yahoo/search/yql/YqlParser.java
+++ b/container-search/src/main/java/com/yahoo/search/yql/YqlParser.java
@@ -707,7 +707,7 @@ public class YqlParser implements Parser {
     }
 
     private Item buildUserInput(OperatorNode<ExpressionOperator> ast) {
-        // TODO add support for default arguments if property results in nothing
+        // TODO: Add support for default arguments if property results in nothing
         List<OperatorNode<ExpressionOperator>> args = ast.getArgument(1);
         String wordData = getStringContents(args.get(0));
 
@@ -727,7 +727,7 @@ public class YqlParser implements Parser {
             item = instantiateWordItem(defaultIndex, wordData, ast, null, SegmentWhen.ALWAYS, false, language);
         } else {
             item = parseUserInput(grammar, defaultIndex, wordData, language, allowEmpty);
-            propagateUserInputAnnotations(ast, item);
+            propagateUserInputAnnotationsRecursively(ast, item);
         }
 
         // Set grammar-specific annotations
@@ -767,7 +767,7 @@ public class YqlParser implements Parser {
         }
     }
 
-    private void propagateUserInputAnnotations(OperatorNode<ExpressionOperator> ast, Item item) {
+    private void propagateUserInputAnnotationsRecursively(OperatorNode<ExpressionOperator> ast, Item item) {
         ToolBox.visit(new AnnotationPropagator(ast), item);
     }