From 184285ab277bd0836b4e5336754ed459339d88c1 Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Wed, 24 Aug 2016 13:01:54 +0200 Subject: [PATCH] Polishing --- .../beans/factory/config/NamedBeanHolder.java | 9 ++++++++- .../org/springframework/web/cors/CorsConfiguration.java | 2 +- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/NamedBeanHolder.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/NamedBeanHolder.java index b950aef98db..04e5e39a372 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/NamedBeanHolder.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/NamedBeanHolder.java @@ -17,6 +17,7 @@ package org.springframework.beans.factory.config; import org.springframework.beans.factory.NamedBean; +import org.springframework.util.Assert; /** * A simple holder for a given bean name plus bean instance. @@ -34,20 +35,26 @@ public class NamedBeanHolder implements NamedBean { /** * Create a new holder for the given bean name plus instance. + * @param beanName the name of the bean + * @param beanInstance the corresponding bean instance */ public NamedBeanHolder(String beanName, T beanInstance) { + Assert.notNull(beanName, "Bean name must not be null"); this.beanName = beanName; this.beanInstance = beanInstance; } + /** + * Return the name of the bean (never {@code null}). + */ @Override public String getBeanName() { return this.beanName; } /** - * Return the corresponding bean instance. + * Return the corresponding bean instance (can be {@code null}). */ public T getBeanInstance() { return this.beanInstance; diff --git a/spring-web/src/main/java/org/springframework/web/cors/CorsConfiguration.java b/spring-web/src/main/java/org/springframework/web/cors/CorsConfiguration.java index 95d1134d7c2..d774012e517 100644 --- a/spring-web/src/main/java/org/springframework/web/cors/CorsConfiguration.java +++ b/spring-web/src/main/java/org/springframework/web/cors/CorsConfiguration.java @@ -286,7 +286,7 @@ public class CorsConfiguration { throw new IllegalArgumentException("'*' is not a valid exposed header value"); } if (this.exposedHeaders == null) { - this.exposedHeaders = new ArrayList(); + this.exposedHeaders = new ArrayList(4); } this.exposedHeaders.add(exposedHeader); }