RFR: 8150823: UL handles disabling logs incorrectly

[Robbin Ehn]

Hi Marcus

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.

I think this looks good.

Thanks for fixing!

/Robbin (not a reviewer as you know)

>
> 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
>