PING: Re: RFR: 8150823: UL handles disabling logs incorrectly

[Marcus Larsson]

Need a Reviewer for this.

Thanks,
Marcus

On 03/23/2016 11:41 AM, Marcus Larsson wrote:
> On 03/18/2016 12:25 PM, Marcus Larsson wrote:
>> Hi,
>>
>> Updated the patch after offline discussion. New patch attempts to 
>> normalize the output name before searching for it or adding it. This 
>> includes adding the implicit file= prefix if missing, and/or removing 
>> quotes from a quoted name.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~mlarsson/8150823/webrev.01/
>>
>> Incremental:
>> http://cr.openjdk.java.net/~mlarsson/8150823/webrev.00-01/
>>
>> Tested with serviceability/logging tests through RBT.
>>
>> Thanks,
>> Marcus
>>
>> On 03/11/2016 03:40 PM, Marcus Larsson wrote:
>>> Hi,
>>>
>>> Please review the following patch to fix a problem with disabling 
>>> log outputs using -Xlog:disable or disable_logging() when there are 
>>> multiple file outputs configured. The patch also resolves an issue 
>>> with log file lookups that could cause multiple outputs to be 
>>> created for the same log file.
>>>
>>> Log output removal now iterates backwards to not invalidate the loop 
>>> index, and file outputs are always prepended with "file=" during 
>>> lookups and listings.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~mlarsson/8150823/webrev.00/
>>>
>>> Issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8150823
>>>
>>> Testing:
>>> Included tests through RBT
>>>
>>> Thanks,
>>> Marcus
>>
>