From 965105b8fe6ae3160aea29ba9b14b06ec8c778a1 Mon Sep 17 00:00:00 2001 From: Adrien Grand Date: Wed, 22 Apr 2015 14:02:58 +0000 Subject: [PATCH] LUCENE-6446: Fix method visibility and trappy factory method. git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/trunk@1675365 13f79535-47bb-0310-9956-ffa450edef68 --- .../org/apache/lucene/search/Explanation.java | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/lucene/core/src/java/org/apache/lucene/search/Explanation.java b/lucene/core/src/java/org/apache/lucene/search/Explanation.java index c554861efaa..9f767ca4277 100644 --- a/lucene/core/src/java/org/apache/lucene/search/Explanation.java +++ b/lucene/core/src/java/org/apache/lucene/search/Explanation.java @@ -25,7 +25,7 @@ import java.util.List; import java.util.Objects; /** Expert: Describes the score computation for document and query. */ -public class Explanation { +public final class Explanation { /** * Create a new explanation for a match. @@ -51,14 +51,14 @@ public class Explanation { * Create a new explanation for a document which does not match. */ public static Explanation noMatch(String description, Collection details) { - return new Explanation(false, 0f, description, Collections.emptyList()); + return new Explanation(false, 0f, description, details); } /** * Create a new explanation for a document which does not match. */ public static Explanation noMatch(String description, Explanation... details) { - return new Explanation(false, 0f, description, Collections.emptyList()); + return new Explanation(false, 0f, description, Arrays.asList(details)); } private final boolean match; // whether the document matched @@ -90,11 +90,7 @@ public class Explanation { /** A description of this explanation node. */ public String getDescription() { return description; } - /** - * A short one line summary which should contain all high level - * information about this Explanation, without the "Details" - */ - protected String getSummary() { + private String getSummary() { return getValue() + " = " + getDescription(); } @@ -108,7 +104,8 @@ public class Explanation { public String toString() { return toString(0); } - protected String toString(int depth) { + + private String toString(int depth) { StringBuilder buffer = new StringBuilder(); for (int i = 0; i < depth; i++) { buffer.append(" ");