RFR: 8307160: [REDO] Enable the permissive- flag on the Microsoft Visual C compiler [v4]

Magnus Ihse Bursie ihse at openjdk.org
Thu Sep 21 09:32:54 UTC 2023


On Thu, 14 Sep 2023 03:23:55 GMT, Julian Waters <jwaters at openjdk.org> wrote:

>> Julian Waters has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Document changes in awt_DnDDS.cpp
>
> Pinging

@TheShermanTanker In my experience, getting reviews from all areas for issues like this that cuts through the entire JDK can be difficult. Another approach, which requires more work from your side, but hopefully less from the reviewers' (and thus makes it easier for them to review) is to split this PR into multiple ones: One for each area (basically, area == mailing list) that just makes the changes to the code necessary to (in the future) turn on /permissive-. And then finally a small "finishing" PR which just touches the makefile and enables the flag, when all code is fixed. 

As a side effect, it is also 100% clear that all parts of the code has been correctly reviewed, since then reviewers do not need to leave conditions on their reviews ("i only looked at the foo parts").

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

PR Comment: https://git.openjdk.org/jdk/pull/15096#issuecomment-1729204060


More information about the client-libs-dev mailing list