You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by si...@apache.org on 2018/09/17 14:05:51 UTC

lucene-solr:master: LUCENE-8503: Call #getDelegate instead of direct member access during unwrap

Repository: lucene-solr
Updated Branches:
  refs/heads/master 36eae5716 -> 44dbfaab1


LUCENE-8503: Call #getDelegate instead of direct member access during unwrap

Filter*Reader instances access the member or the delegate directly instead of
calling getDelegate(). In order to track access of the delegate these methods
should call #getDelegat()


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/44dbfaab
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/44dbfaab
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/44dbfaab

Branch: refs/heads/master
Commit: 44dbfaab19b5845f27d6174071bd3808756ff552
Parents: 36eae57
Author: Simon Willnauer <si...@apache.org>
Authored: Mon Sep 17 10:39:41 2018 +0200
Committer: Simon Willnauer <si...@apache.org>
Committed: Mon Sep 17 16:05:26 2018 +0200

----------------------------------------------------------------------
 .../core/src/java/org/apache/lucene/index/FilterCodecReader.java | 4 ++--
 .../src/java/org/apache/lucene/index/FilterDirectoryReader.java  | 2 +-
 .../core/src/java/org/apache/lucene/index/FilterLeafReader.java  | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/44dbfaab/lucene/core/src/java/org/apache/lucene/index/FilterCodecReader.java
----------------------------------------------------------------------
diff --git a/lucene/core/src/java/org/apache/lucene/index/FilterCodecReader.java b/lucene/core/src/java/org/apache/lucene/index/FilterCodecReader.java
index 0f8dcfc..ff3ea18 100644
--- a/lucene/core/src/java/org/apache/lucene/index/FilterCodecReader.java
+++ b/lucene/core/src/java/org/apache/lucene/index/FilterCodecReader.java
@@ -48,8 +48,8 @@ public abstract class FilterCodecReader extends CodecReader {
     }
     return reader;
   }
-  /** 
-   * The underlying CodecReader instance. 
+  /**
+   * The underlying CodecReader instance.
    */
   protected final CodecReader in;
   

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/44dbfaab/lucene/core/src/java/org/apache/lucene/index/FilterDirectoryReader.java
----------------------------------------------------------------------
diff --git a/lucene/core/src/java/org/apache/lucene/index/FilterDirectoryReader.java b/lucene/core/src/java/org/apache/lucene/index/FilterDirectoryReader.java
index 4a9cd68..70ecd12 100644
--- a/lucene/core/src/java/org/apache/lucene/index/FilterDirectoryReader.java
+++ b/lucene/core/src/java/org/apache/lucene/index/FilterDirectoryReader.java
@@ -37,7 +37,7 @@ public abstract class FilterDirectoryReader extends DirectoryReader {
    *  an instance of {@link FilterDirectoryReader}.  */
   public static DirectoryReader unwrap(DirectoryReader reader) {
     while (reader instanceof FilterDirectoryReader) {
-      reader = ((FilterDirectoryReader) reader).in;
+      reader = ((FilterDirectoryReader) reader).getDelegate();
     }
     return reader;
   }

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/44dbfaab/lucene/core/src/java/org/apache/lucene/index/FilterLeafReader.java
----------------------------------------------------------------------
diff --git a/lucene/core/src/java/org/apache/lucene/index/FilterLeafReader.java b/lucene/core/src/java/org/apache/lucene/index/FilterLeafReader.java
index 8e1f272..418e7a8 100644
--- a/lucene/core/src/java/org/apache/lucene/index/FilterLeafReader.java
+++ b/lucene/core/src/java/org/apache/lucene/index/FilterLeafReader.java
@@ -44,7 +44,7 @@ public abstract class FilterLeafReader extends LeafReader {
    *  an instance of {@link FilterLeafReader}.  */
   public static LeafReader unwrap(LeafReader reader) {
     while (reader instanceof FilterLeafReader) {
-      reader = ((FilterLeafReader) reader).in;
+      reader = ((FilterLeafReader) reader).getDelegate();
     }
     return reader;
   }