RFR(S): 8130036: Fix problems with imprecise C++ coding.

David Holmes david.holmes at oracle.com
Mon Jun 29 09:26:00 UTC 2015


Hi Goetz,

Good catches! If no-one steps in overnight I'll take care of this in my 
morning.

Thanks,
David

On 29/06/2015 6:30 PM, Lindenmaier, Goetz wrote:
> Hi,
>
>
>
> We detected some minor, but obviously wrong coding issues we would like to fix.
>
>
>
> In two places {} are missing after an if, whilst there are two statements that depend
>
> on the condition (CHECK macro expands to another statement).
>
>
>
> In compilerOracle.cpp, a wrong size is passed to jio_snprintf().
>
>
>
> Please review this change.  I please need a sponsor.
>
> http://cr.openjdk.java.net/~goetz/webrevs/8130036_blocks/webrev.01/8130036_blocks-hs-rt.changeset
>
>
>
> Best regards,
>
>    Goetz.
>
> Sorry for the resend, the subject was missing ...
>


More information about the hotspot-dev mailing list