On Thu, 27 Jul 2023 19:37:33 GMT, Phil Race <[email protected]> wrote:
>> Aleksey Shipilev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Disable the warnings instead > > make/modules/java.desktop/lib/Awt2dLibraries.gmk line 248: > >> 246: DISABLED_WARNINGS_gcc_OGLBufImgOps.c := format-nonliteral, \ >> 247: DISABLED_WARNINGS_gcc_OGLPaints.c := format-nonliteral, \ >> 248: DISABLED_WARNINGS_gcc_screencast_pipewire.c := undef, \ > > At least ignoring it just for the specific compilation units is not too bad. > > I'll approve but I'd like to see a comment here as to why its needed - ie > just for gcc6 so people have an idea when we might be able to remove it and > know not to remove it because it doesn't matter for the current compiler > version. > > I've still asked upstream to consider applying your first version so that > we'd not need this although perhaps it won't matter for long .. once no one > cares about gcc6 because we need to move on for other reasons. There is no good way to put the comment straight into this block, AFAICS. Can we settle on repository history to tell future investigations why this was added? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14995#discussion_r1277221303
