On Mon, 23 Aug 2021 18:40:55 GMT, Johan Vos <j...@openjdk.org> wrote:

>> OK, that seems reasonable, and I can't think of any downside.
>
> The only thing that makes me slightly nervous is seeing that this is needed 
> on a number of places, leading to more code duplication (and the build.gradle 
> is already rather large). But that is not a problem introduced with this PR, 
> so I'm not requesting a change for this.

Yeah. At least, the four duplicated places look very similar to each other. 
Perhaps, those lines (adding module-related compiler args) can be extracted 
into a dedicated method. But that should be done, if at all, in a different 
task/PR.

-------------

PR: https://git.openjdk.java.net/jfx/pull/608

Reply via email to