RFR: 8244681: Add a warning for possibly lossy conversion in compound assignments [v3]
Magnus Ihse Bursie
ihse at openjdk.java.net
Wed May 11 13:13:43 UTC 2022
On Wed, 11 May 2022 13:05:45 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:
>> make/modules/jdk.jfr/Java.gmk line 26:
>>
>>> 24: #
>>> 25:
>>> 26: DISABLED_WARNINGS_java += exports lossy-conversions
>>
>> Note that with the fix of JDK-8286392 (and JDK-8286396) the `lossy-conversions` warning should not be disabled for the JFR code.
>>
>> In general, you need to check which of the subtasks of JDK-8286374 that has been fixed, and adjust the makefiles accordingly, before pushing this fix.
>>
>> (In the future, it might be easier to push the fix which disables the warnings first, and then file follow-up bugs on aa per-component basis, and remind them to remove the disabling in the makefile. That way there won't be a race between individual fixes and a "master" bug like this.)
>
> I agree, but if it doesn't happen, I can follow up with a separate PR where I remove the disablement.
That's good to know. I think the tricky part is mostly about keeping track of all these disabled warnings, so they are not kept around longer than necessary. And that needs coordination with all the subtasks of the umbrella issue.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8599
More information about the compiler-dev
mailing list