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

[Marcus Larsson]

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
>