CompilierConfiguration#CURRENT_JVM_VERSION

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

CompilierConfiguration#CURRENT_JVM_VERSION

paulk_asert

Hi all,

In CompilerConfiguration we have a public constant CURRENT_JVM_VERSION which actually represents the minimum JDK version supported by a particular version of Groovy (potentially on a particular VM runtime). I was thinking of deprecating above constant (it's not used in the Groovy codebase) and renaming to MINIMUM_JDK_VERSION.

If you prefer a different name or have any other comments, let me know.

Cheers, Paul.

Reply | Threaded
Open this post in threaded view
|

Re: CompilierConfiguration#CURRENT_JVM_VERSION

Daniel.Sun
Reply | Threaded
Open this post in threaded view
|

Re: CompilierConfiguration#CURRENT_JVM_VERSION

jwagenleitner
In reply to this post by paulk_asert
+1 for deprecating the current constant and new name (see note below).

If the new constant wont be used internally then I might suggest no new constant is created. Else if it's created in order to provide an alternate to the deprecated one, would it make sense to provide it via `groovy.lang.GroovySystem` or some other public api class outside of the internal org. package structure?

On Tue, May 15, 2018 at 3:01 AM Paul King <[hidden email]> wrote:

Hi all,

In CompilerConfiguration we have a public constant CURRENT_JVM_VERSION which actually represents the minimum JDK version supported by a particular version of Groovy (potentially on a particular VM runtime). I was thinking of deprecating above constant (it's not used in the Groovy codebase) and renaming to MINIMUM_JDK_VERSION.

If you prefer a different name or have any other comments, let me know.

Cheers, Paul.

Reply | Threaded
Open this post in threaded view
|

Re: CompilierConfiguration#CURRENT_JVM_VERSION

Remko Popma
In reply to this post by paulk_asert
+1

On Tue, May 15, 2018 at 7:01 PM, Paul King <[hidden email]> wrote:

Hi all,

In CompilerConfiguration we have a public constant CURRENT_JVM_VERSION which actually represents the minimum JDK version supported by a particular version of Groovy (potentially on a particular VM runtime). I was thinking of deprecating above constant (it's not used in the Groovy codebase) and renaming to MINIMUM_JDK_VERSION.

If you prefer a different name or have any other comments, let me know.

Cheers, Paul.