Address some review comments:
authormarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Sun, 5 Dec 2010 19:16:07 +0000 (19:16 +0000)
committermarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Sun, 5 Dec 2010 19:16:07 +0000 (19:16 +0000)
commit71b49610cdfd0d4461517a30cc02f3e88386245f
tree266ca13eb219965a8b2c91a96ae3b3dc71147079
parent791dab2e77518b1e63e3be008e6d7b210c96d035
Address some review comments:
1. Avoid possible NPEs by using a local variable for the newDefaultAccessLog
2. PropertyChangeListener should be added to the Engine
3. Add the listener when the Noop logger is used so changes can be detected that could cause a different logger to be used.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1042413 13f79535-47bb-0310-9956-ffa450edef68
java/org/apache/catalina/core/StandardEngine.java