You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by jp...@apache.org on 2024/01/29 08:19:31 UTC

(lucene) branch branch_9x updated: Make use of null-checked variable (#13040)

This is an automated email from the ASF dual-hosted git repository.

jpountz pushed a commit to branch branch_9x
in repository https://gitbox.apache.org/repos/asf/lucene.git


The following commit(s) were added to refs/heads/branch_9x by this push:
     new 5a54a97be3e Make use of null-checked variable (#13040)
5a54a97be3e is described below

commit 5a54a97be3e4824a6afc2798251c03ee705c3d1b
Author: sabi0 <2s...@gmail.com>
AuthorDate: Mon Jan 29 09:19:07 2024 +0100

    Make use of null-checked variable (#13040)
---
 lucene/demo/src/java/org/apache/lucene/demo/SearchFiles.java           | 2 +-
 .../apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java   | 3 +--
 .../lucene/luke/models/util/twentynewsgroups/MessageFilesParser.java   | 2 +-
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/lucene/demo/src/java/org/apache/lucene/demo/SearchFiles.java b/lucene/demo/src/java/org/apache/lucene/demo/SearchFiles.java
index 67fcd762be1..17c2d6b1e6e 100644
--- a/lucene/demo/src/java/org/apache/lucene/demo/SearchFiles.java
+++ b/lucene/demo/src/java/org/apache/lucene/demo/SearchFiles.java
@@ -217,7 +217,7 @@ public class SearchFiles {
           System.out.println((i + 1) + ". " + path);
           String title = doc.get("title");
           if (title != null) {
-            System.out.println("   Title: " + doc.get("title"));
+            System.out.println("   Title: " + title);
           }
         } else {
           System.out.println((i + 1) + ". " + "No path for this document");
diff --git a/lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java b/lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java
index f167bf3b68b..7ec8fe416a6 100644
--- a/lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java
+++ b/lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java
@@ -300,8 +300,7 @@ public class SortedSetDocValuesFacetCounts extends AbstractSortedSetDocValueFace
       if (liveDocs == null) {
         countOneSegmentNHLD(ordinalMap, context.reader(), context.ord);
       } else {
-        countOneSegment(
-            ordinalMap, context.reader(), context.ord, null, context.reader().getLiveDocs());
+        countOneSegment(ordinalMap, context.reader(), context.ord, null, liveDocs);
       }
     }
   }
diff --git a/lucene/luke/src/java/org/apache/lucene/luke/models/util/twentynewsgroups/MessageFilesParser.java b/lucene/luke/src/java/org/apache/lucene/luke/models/util/twentynewsgroups/MessageFilesParser.java
index 41db69b2f18..fd9d1286769 100644
--- a/lucene/luke/src/java/org/apache/lucene/luke/models/util/twentynewsgroups/MessageFilesParser.java
+++ b/lucene/luke/src/java/org/apache/lucene/luke/models/util/twentynewsgroups/MessageFilesParser.java
@@ -54,7 +54,7 @@ public class MessageFilesParser extends SimpleFileVisitor<Path> {
       if (attr.isRegularFile()) {
         Message message = parse(file);
         if (message != null) {
-          messages.add(parse(file));
+          messages.add(message);
         }
       }
     } catch (IOException e) {