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

2016-03-30 Thread 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

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

2016-03-23 Thread 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:

Re: RFR: 8150823: UL handles disabling logs incorrectly

2016-03-21 Thread Marcus Larsson
Thanks for reviewing, Robbin! Marcus On 03/21/2016 02:36 PM, Robbin Ehn wrote: 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

Re: RFR: 8150823: UL handles disabling logs incorrectly

2016-03-21 Thread 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.

Re: RFR: 8150823: UL handles disabling logs incorrectly

2016-03-18 Thread Marcus Larsson
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:

RFR: 8150823: UL handles disabling logs incorrectly

2016-03-11 Thread Marcus Larsson
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