RFR(XS): 8137260: fix warning after "8046148: JEP 158: Unified JVM Logging"

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Tue Sep 29 11:14:43 UTC 2015


Hi Markus,

Thanks for sponsoring!

Best regards,
  Goetz.

-----Original Message-----
From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] On Behalf Of Marcus Larsson
Sent: Tuesday, September 29, 2015 12:38 PM
To: hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR(XS): 8137260: fix warning after "8046148: JEP 158: Unified JVM Logging"

Hi,

On 2015-09-29 12:01, Lindenmaier, Goetz wrote:
> Thanks, Thomas!
>
> Best regards,
>    Goetz.
>
> From: Thomas Stüfe [mailto:thomas.stuefe at gmail.com]
> Sent: Tuesday, September 29, 2015 11:24 AM
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> Cc: hotspot-runtime-dev at openjdk.java.net
> Subject: Re: RFR(XS): 8137260: fix warning after "8046148: JEP 158: Unified JVM Logging"
>
> Hi Goetz,
>
> looks good.
>
> Regards, Thomas
>
> On Mon, Sep 28, 2015 at 1:52 PM, Lindenmaier, Goetz <goetz.lindenmaier at sap.com<mailto:goetz.lindenmaier at sap.com>> wrote:
> Hi,
>
> we see warning
> "negative integer implicitly converted to unsigned type"
> for
> size_t first_pos = -1;
> with gcc 4.1.2.
>
> I fixed this by using SIZE_MAX.
>
> Please review this tiny change.  I please need a sponsor.
> http://cr.openjdk.java.net/~goetz/webrevs/8137260-fixWarn/webrev.00/
>
>
> Thanks and best regards,
>    Goetz.
>

Looks good, I will sponsor it.

Thanks,
Marcus


More information about the hotspot-runtime-dev mailing list