groovy-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Guillaume Laforge <>
Subject Re: Cleanup and internal package names
Date Wed, 11 Nov 2015 19:30:56 GMT
And I imagine all the tests are still fine without those internal public
methods removed?

On Wed, Nov 11, 2015 at 7:34 PM, John Wagenleitner <> wrote:

> I was recently looking into a few JSON related issues
> (GROOVY-6813/6922/7665) and came across a number of unused methods in the
> class groovy.json.internal.CharScanner.  Before I did anything I thought
> I'd better check and see if the project team is open to "cleanup" pull
> requests (not associated with any particular issue).
> I know that the stability of the public API is extremely important and
> this class and it's methods are public, but not used outside its module in
> the groovy codebase.  Is being in a package named "internal" (nested within
> a public package groovy.json) sufficient to allow removing unused public
> methods and/or changing their signatures?
> John

Guillaume Laforge
Apache Groovy committer & PMC member
Product Ninja & Advocate at Restlet <>

Social: @glaforge <> / Google+

View raw message