Config script is for Groovy power users. I think the idea behind this
feature would be to enable everyone to use these often used annotations
without any hassle in a Groovy script (IDEs will insert the import
automatically anyway), as to encourage their use.
Enabling static compilation by default would be a good idea imho (since
it catches some trivial errors, and it is harder to go from dynamic to
static later on), but the static compiler would need to not reject valid
constructs without one needing to resort to explicit casts (or falling
back to @CompileDynamic) first...
On 28/10/2019 16:28, Milles, Eric (TR Tech, Content & Ops) wrote:
I agree, my general opinion is that there is already the config script
mechanism to either 1) add imports, 2) add star imports or 3) enable
specific AST transforms without the need for annotating sources.
There was a change a few months back to try and enable static
compilation by default and it was quickly removed (commented out).
------------------------------------------------------------------------
*From:* Guillaume Laforge <glafo...@gmail.com>
*Sent:* Monday, October 28, 2019 9:57 AM
*To:* Groovy_Developers <dev@groovy.apache.org>
*Subject:* Re: [GEP] Import `TypeChecked`, `CompileStatic` and
`CompileDynamic` by default
Interestingly, historically, we've gone the other way round, to move
default-imported transforms in non-default-imported packages,
like @Delegate, and others.
Also for those who frequently use these transformations, there's also
the option of defining a compiler configuration script that can be
passed as argument to the compiler.
(I'm neither saying no nor yes, just stating some facts) :-)
On Mon, Oct 28, 2019 at 3:53 PM Milles, Eric (TR Tech, Content & Ops)
<eric.mil...@thomsonreuters.com
<mailto:eric.mil...@thomsonreuters.com>> wrote:
One way for them to be "imported by default" would be to move them
to the groovy.lang package. I'd make sure ClosureParams and
DelegatesTo come along for the ride if such a change is made.
------------------------------------------------------------------------
*From:* Daniel.Sun <sun...@apache.org <mailto:sun...@apache.org>>
*Sent:* Sunday, October 27, 2019 9:39 PM
*To:* d...@groovy.incubator.apache.org
<mailto:d...@groovy.incubator.apache.org>
<d...@groovy.incubator.apache.org
<mailto:d...@groovy.incubator.apache.org>>
*Subject:* Re: [GEP] Import `TypeChecked`, `CompileStatic` and
`CompileDynamic` by default
Hi mg,
Groovy 3 is already in RC phase, so its features are frozen. New
features are targeted to next version.
Cheers,
Daniel.Sun
-----
Apache Groovy committer & PMC member
Blog:
https://urldefense.proofpoint.com/v2/url?u=http-3A__blog.sunlan.me&d=DwICAg&c=4ZIZThykDLcoWk-GVjSLmy8-1Cr1I4FWIvbLFebwKgY&r=tPJuIuL_GkTEazjQW7vvl7mNWVGXn3yJD5LGBHYYHww&m=r-PbGpewipkWfq0YTbHtl7RGQlmtXjTW5-QGZheKe44&s=EXzjCAeqXoEMlf_PTTobs1M-RmigD_yhBfLteQ1LFf0&e=
Twitter: @daniel_sun
--
Sent from:
https://urldefense.proofpoint.com/v2/url?u=http-3A__groovy.329449.n5.nabble.com_Groovy-2DDev-2Df372993.html&d=DwICAg&c=4ZIZThykDLcoWk-GVjSLmy8-1Cr1I4FWIvbLFebwKgY&r=tPJuIuL_GkTEazjQW7vvl7mNWVGXn3yJD5LGBHYYHww&m=r-PbGpewipkWfq0YTbHtl7RGQlmtXjTW5-QGZheKe44&s=gmdm74xk2CtbHj0eVWzHqXWN8gJXw9jEHmlDwAgjcvY&e=
--
Guillaume Laforge
Apache Groovy committer
Developer Advocate @ Google Cloud Platform
Blog: http://glaforge.appspot.com/
<https://urldefense.proofpoint.com/v2/url?u=http-3A__glaforge.appspot.com_&d=DwMFaQ&c=4ZIZThykDLcoWk-GVjSLmy8-1Cr1I4FWIvbLFebwKgY&r=tPJuIuL_GkTEazjQW7vvl7mNWVGXn3yJD5LGBHYYHww&m=kwm82EmdsJrZlY7LZZQ4BTlb7NC2L-05IL03Hil7Roc&s=4XxsNH60q7qLdmxG0Qo0hoMOKNui89wQUyOlgNkG2ks&e=>
Twitter: @glaforge
<https://urldefense.proofpoint.com/v2/url?u=http-3A__twitter.com_glaforge&d=DwMFaQ&c=4ZIZThykDLcoWk-GVjSLmy8-1Cr1I4FWIvbLFebwKgY&r=tPJuIuL_GkTEazjQW7vvl7mNWVGXn3yJD5LGBHYYHww&m=kwm82EmdsJrZlY7LZZQ4BTlb7NC2L-05IL03Hil7Roc&s=RxHGbRgyFUnaOtQDkcvefRQKGhAwkH6pRMkDpPxId7M&e=>
