Vincent, Seems like the appropriate thing to do would just be to remove the variable and the constructor, since neither existed in Lucene anyway. If someone wanted to make a subclass and they were interested in the "outer Directory", they could just add their own constructor, but it doesn't seem practical to force every subclass to provide a constructor if they don't actually need it to populate a variable that is not even used.
Would you like to submit a PR for the change? Thanks, Shad Storhaug (NightOwl888) -----Original Message----- From: Van Den Berghe, Vincent [mailto:[email protected]] Sent: Thursday, July 13, 2017 2:04 PM To: [email protected] Subject: A request Hey guys, The IndexInputSlicer class is a public abstract class with a private member that isn't used anywhere: public abstract class IndexInputSlicer : IDisposable { private readonly Directory outerInstance; It is useless by definition (it's inaccessible). In recent Java version, there is no such outerInstance. It's handy in .NET because sometimes we need explicit access to outer instances (which is silently allowed in Java), so could this be made at least protected? Vincent
