RFR JDK-8068448 - improve the assert
Stefan Karlsson
stefan.karlsson at oracle.com
Wed Apr 8 07:43:33 UTC 2015
On 2015-04-07 22:48, Bengt Rutisson wrote:
>
> Hi Eric,
>
> On 07/04/15 20:19, Eric Caspole wrote:
>> Hi everybody,
>> Here is a patch to try to help debug this problem, by explicitly
>> collecting more information including start array and heap size when
>> the assert occurs.
>>
>> http://cr.openjdk.java.net/~ecaspole/JDK-8068448/00/webrev/
>
> The change looks good to me.
Looks good to me to.
Style-wise I would prefer an extra blank line before the code section
you added (between line 154 and 155).
Thanks,
StefanK
>
> However, I think you should use a separate bug number to track this
> change. JDK-8068448 should probably be used when you push the actual
> fix for the issue.
>
> Thanks,
> Bengt
>
>>
>> Passes JPRT.
>> Thanks,
>> Eric
>>
>
More information about the hotspot-gc-dev
mailing list