Polish

Author: Phillip Webb

spring-boot/src/main/java/org/springframework/boot/builder/ParentContextCloserApplicationListener.java

@@ -26,6 +26,7 @@
 import org.springframework.context.ConfigurableApplicationContext;
 import org.springframework.context.event.ContextClosedEvent;
 import org.springframework.core.Ordered;
+import org.springframework.util.ObjectUtils;
 
 /**
  * Listener that closes the application context if its parent is closed. It listens for
@@ -77,6 +78,9 @@ protected ContextCloserListener createContextCloserListener(
 		return new ContextCloserListener(child);
 	}
 
+	/**
+	 * {@link ApplicationListener} to close the context.
+	 */
 	protected static class ContextCloserListener implements
 			ApplicationListener<ContextClosedEvent> {
 
@@ -99,31 +103,23 @@ public void onApplicationEvent(ContextClosedEvent event) {
 
 		@Override
 		public int hashCode() {
-			final int prime = 31;
-			int result = 1;
-			result = prime
-					* result
-					+ ((this.childContext.get() == null) ? 0 : this.childContext.get()
-							.hashCode());
-			return result;
+			return ObjectUtils.nullSafeHashCode(this.childContext.get());
 		}
 
 		@Override
 		public boolean equals(Object obj) {
-			if (this == obj)
+			if (this == obj) {
 				return true;
-			if (obj == null)
-				return false;
-			if (getClass() != obj.getClass())
-				return false;
-			ContextCloserListener other = (ContextCloserListener) obj;
-			if (this.childContext.get() == null) {
-				if (other.childContext.get() != null)
-					return false;
 			}
-			else if (!this.childContext.get().equals(other.childContext.get()))
+			if (obj == null) {
 				return false;
-			return true;
+			}
+			if (obj instanceof ContextCloserListener) {
+				ContextCloserListener other = (ContextCloserListener) obj;
+				return ObjectUtils.nullSafeEquals(this.childContext.get(),
+						other.childContext.get());
+			}
+			return super.equals(obj);
 		}
 
 	}

spring-boot/src/main/java/org/springframework/boot/context/event/EventPublishingRunListener.java

@@ -90,10 +90,9 @@ public void contextLoaded(ConfigurableApplicationContext context) {
 	@Override
 	public void finished(ConfigurableApplicationContext context, Throwable exception) {
 		if (exception != null) {
-			publishEvent(new ApplicationFailedEvent(this.application, this.args, context,
-					exception));
-		}
-		else {
+			ApplicationFailedEvent event = new ApplicationFailedEvent(this.application,
+					this.args, context, exception);
+			publishEvent(event);
 		}
 	}
 

spring-boot/src/test/java/org/springframework/boot/SpringApplicationTests.java

@@ -80,6 +80,7 @@
  * Tests for {@link SpringApplication}.
  * 
  * @author Phillip Webb
+ * @author Dave Syer
  */
 public class SpringApplicationTests {