On Wed, 11 May 2022 22:03:38 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:

> It would not make sense to set the disabled warning in the configure script, 
> no. The current code looks perfectly fine. Disabled warnings per module are 
> set in the makefiles.

OK, that you for your feedback regarding the makefile changes vs. configure 
script. 
> 
> If you feel strongly that this needs to be documented more than in the JBS 
> issue and this PR review, updating the zlib ChangeLog file is probably the 
> way to go. I have no strong opinion on that. But from the build system PoV, 
> the current code is fine as it is to be committed.


OK, I am probably overthinking the need to document this

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

PR: https://git.openjdk.java.net/jdk/pull/8651

Reply via email to