RFR (L): 8062370: Various minor code improvements
Thomas Stüfe
thomas.stuefe at gmail.com
Fri Nov 14 09:59:49 UTC 2014
Hi David,
thanks for looking. See here the corrected webrev:
http://cr.openjdk.java.net/~simonis/webrevs/8064779/
for this new bug report:
https://bugs.openjdk.java.net/browse/JDK-8064779
Best Regards, Thomas
On Thu, Nov 13, 2014 at 4:03 AM, David Holmes <david.holmes at oracle.com>
wrote:
> Hi Thomas,
>
> On 12/11/2014 8:31 PM, Thomas Stüfe wrote:
>
>> Hi,
>>
>> could you please review this little addition? (added comments for
>> jio_snprintf)
>>
>> http://cr.openjdk.java.net/~simonis/webrevs/8062370/
>>
>
> A new bug is needed for these changes.
>
> As people rarely look at the header file when reading the code could you
> augment the last line of the comment in jvm.cpp from:
>
> + // return always -1.
>
> to
>
> + // always return -1, and perform null termination.
>
> Thanks,
> David
>
>
More information about the hotspot-dev
mailing list