RFR: 8196988 (Resolve disabled warnings for libjimage)
Srinivas Dama
srinivas.dama at oracle.com
Mon Jun 18 02:35:59 UTC 2018
Thank you Paul for the review.
>I am guessing the comment selectively disables the warning in this code (rather than using a diagnostic pragma).
yes
I have updated the patch on latest repo changes.
http://cr.openjdk.java.net/~sdama/8196988/webrev.02/
pushing the above changeset.
Note: I have run mach5 tests successfully.
Regards,
Srinivas
----- Original Message -----
From: paul.sandoz at oracle.com
To: srinivas.dama at oracle.com
Cc: core-libs-dev at openjdk.java.net, james.laskey at oracle.com
Sent: Tuesday, 12 June, 2018 10:08:53 PM GMT +05:30 Chennai, Kolkata, Mumbai, New Delhi
Subject: Re: RFR: 8196988 (Resolve disabled warnings for libjimage)
Looks ok. Including Jim in case he is not listening in…
I am guessing the comment selectively disables the warning in this code (rather than using a diagnostic pragma).
Thanks,
Paul.
> On Jun 7, 2018, at 11:17 PM, Srinivas Dama <srinivas.dama at oracle.com> wrote:
>
> Hi,
>
> Please review http://cr.openjdk.java.net/~sdama/8196988/webrev.01/
> for https://bugs.openjdk.java.net/browse/JDK-8196988
>
>
> Note: Modified earlier fix to handle warning messages specific to
> libjimage only.
>
> Regards,
> Srinivas
>
> ----- Original Message -----
> From: srinivas.dama at oracle.com
> To: core-libs-dev at openjdk.java.net
> Sent: Monday, 4 June, 2018 11:08:08 PM GMT +05:30 Chennai, Kolkata, Mumbai, New Delhi
> Subject: RFR: 8196988 (Resolve disabled warnings for implicit-fallthrough gcc option)
>
> Hi,
>
> Please review http://cr.openjdk.java.net/~sdama/8196988/webrev.00/
> for https://bugs.openjdk.java.net/browse/JDK-8196988
>
> Note: gcc 7.3 compiler emits warnings for implicit fall-through switch cases,but
> these warnings are disabled earlier.I have enabled these warnings and fixed in sources.
>
> Regards,
> Srinivas
More information about the core-libs-dev
mailing list