RFR: 8148755: -XX:+HeapDumpAfterFullGC creates heap dump both before and after Full GC

[sangheon]

Hi Claes,

On 02/02/2016 04:04 PM, Claes Redestad wrote:
> Hi Sangheon,
>
> On 2016-02-03 00:05, sangheon wrote:
>> Hi Claes,
>>
>> Looks good 
>
> thanks!
>
>> and I have minor comments.
>>
>> 1. Line 598: 'before' should be 'after'.
>>
>>  597 void CollectedHeap::post_full_gc_dump(GCTimer* timer) {
>>  598 full_gc_dump(timer, false /* before */);
>>  599 }
>
> "before" here is the name of the parameter, which I thought was the 
> convention when calling out parameter values.
I see and read your next email as well that you will remove this comment.

Thanks,
Sangheon


>
>>
>>
>> 2. Copyright year update.
>
> Deal!
>
> /Claes
>
>>
>> Thanks,
>> Sangheon
>>
>>
>> On 02/02/2016 07:02 AM, Claes Redestad wrote:
>>> Hello,
>>>
>>> this patch fixes an issue where heap is dumped both before and after 
>>> full GC if either one of -XX:+HeapDumpAfterFullGC or 
>>> -XX:+HeapDumpBeforeFullGC is specified.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8148755
>>> Webrev: http://cr.openjdk.java.net/~redestad/8148755/webrev.01
>>>
>>> Testing: JPRT, manual verification
>>>
>>> Thanks!
>>>
>>> /Claes
>>
>