lucenenet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Shad Storhaug <s...@shadstorhaug.com>
Subject RE: A request
Date Thu, 13 Jul 2017 08:09:36 GMT
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:Vincent.VanDenBerghe@bvdinfo.com] 
Sent: Thursday, July 13, 2017 2:04 PM
To: dev@lucenenet.apache.org
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



Mime
View raw message