Daniel,
Looks good. As always, thanks for fixing this.
Jason
----------------------------------------
> Date: Fri, 10 Oct 2014 17:39:55 +0200
> From: daniel.fu...@oracle.com
> To: stani...@riflexo.com; core-libs-dev@openjdk.java.net
> CC: jason_mehr...@hotmail.com
> Subject: Re: RFR: 8060132: Handlers configured on abstract nodes in
> logging.properties are not always properly closed
>
> On 10/10/14 17:10, Stanimir Simeonoff wrote:
>> persistentLoggers should be cleared on reset(), imo.
>
> Yes - I was wondering about that too, it's a bit tricky to
> get it right (WRT MT-safety) :-(.
>
>> Also using "count" and then "for in" to loop over an array looks a bit
>> ugly, should be just "for (int i=0; i<names.length; i++){ final String
>> className = names[i];..}
>
> I wanted to add the logger only if at least one handler was
> successfully created and added, so 'count' is not necessarily
> equals to 'i'.
>
>> Calling the class name 'word' is weird as well. (I do understand that's
>> not new but it's a good time to get it straight).
>
> Agreed.
>
> http://cr.openjdk.java.net/~dfuchs/webrev_8060132/webrev.01
>
> best regards, and thanks for all the valuable inputs & comment from
> you and Jason - I really appreciate it :-)
>
> -- daniel
>
>>
>> Cheers
>> Stanimir
>>
>>
>> On Fri, Oct 10, 2014 at 5:51 PM, Daniel Fuchs <daniel.fu...@oracle.com
>> <mailto:daniel.fu...@oracle.com>> wrote:
>>
>> Hi,
>>
>> Please find below a possible fix for:
>>
>> 8060132: Handlers configured on abstract nodes in logging.properties
>> are not always properly closed
>> https://bugs.openjdk.java.net/__browse/JDK-8060132
>> <https://bugs.openjdk.java.net/browse/JDK-8060132>
>>
>> webrev:
>> http://cr.openjdk.java.net/~__dfuchs/webrev_8060132/webrev.__00/
>> <http://cr.openjdk.java.net/~dfuchs/webrev_8060132/webrev.00/>
>>
>> Other options have been discussed in this other email thread:
>>
>> Subject: JDK-6774110 lock file is not deleted when child logger is used
>> http://mail.openjdk.java.net/__pipermail/core-libs-dev/2014-__October/029038.html
>> <http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-October/029038.html>
>>
>> best regards,
>>
>> -- daniel
>>
>>
>
