On 05/05/15 08:19, Mandy Chung wrote:
On 5/4/2015 11:52 AM, Daniel Fuchs wrote:
On 04/05/15 16:46, Peter Levart wrote:
Hi Daniel,
Here it is:
http://cr.openjdk.java.net/~plevart/misc/LogManager.synchronization/webrev.04/
Looks good for me Peter :-)
Hopefully Mandy will like it too!
Yes it looks good to me. Thanks to both of you for imprpving the
synchronization. This is the comment on the test I sent last round:
TestConfigurationLock.java
Copyright year should be 2015
TestConfigurationLock.properties
It would be good to delete all commented lines except the lines
relevant to the setting to make it obvious what the configuration is.
Right. Sorry I forgot about that. Here is an updated webrev
containing Peter's webrev.04 in which I have updated the test:
http://cr.openjdk.java.net/~dfuchs/webrev_8077846/webrev.05
Peter, are you still willing to push that?
best regards,
-- daniel
Mandy
