RFR: 8043306 - Provide a replacement for the API that allowed to listen for LogManager configuration changes

[Daniel Fuchs]

Hi,

I updated the webrev at 
http://cr.openjdk.java.net/~dfuchs/webrev_8043306/webrev.07

      * @implNote If more than one listener terminates with an uncaught 
error or
      * exception, an implementation may record the additional errors or
      * exceptions as {@linkplain 
Throwable#addSuppressed(java.lang.Throwable)
      * suppressed exceptions}.

best regards,

-- daniel

On 16/09/14 17:31, Daniel Fuchs wrote:
> Done.
>
> http://cr.openjdk.java.net/~dfuchs/webrev_8043306/webrev.06
>
> On 9/16/14 3:56 PM, Alan Bateman wrote:
>> On 15/09/2014 17:03, Daniel Fuchs wrote:
>>> :
>>>
>>>
>>> Here is a new webrev - with updated test case:
>>>
>>> http://cr.openjdk.java.net/~dfuchs/webrev_8043306/webrev.05/
>>>
>> I think this quite good now.
>>
>> Just on this wording:
>>
>> "If a listener terminates with an uncaught error or exception thenthe
>> first exception that was raised will be propagated to the caller of
>> readConfiguration after all listeners have been invoked."
>>
>> I think you can shorten this a bit by dropping "that was raised" from
>> the sentence.
>>
>> Future maintainers might wonder why invokeConfigurationListeners takes a
>> snapshot of the listeners so I think that deserves a comment in the code.
>>
>> -Alan
>