Browse Source

Avoid creating message arguments to Assert.isABC calls

See 67f184293b
pull/1681/head
Vladimir Sitnikov 8 years ago committed by Juergen Hoeller
parent
commit
659f13be1c
  1. 2
      spring-context-support/src/main/java/org/springframework/cache/jcache/interceptor/CacheResolverAdapter.java
  2. 5
      spring-core/src/main/java/org/springframework/core/ParameterizedTypeReference.java
  3. 2
      spring-core/src/main/java/org/springframework/util/xml/TransformerUtils.java
  4. 2
      spring-messaging/src/main/java/org/springframework/messaging/converter/AbstractMessageConverter.java
  5. 2
      spring-messaging/src/main/java/org/springframework/messaging/handler/annotation/support/HeadersMethodArgumentResolver.java
  6. 2
      spring-messaging/src/main/java/org/springframework/messaging/handler/invocation/HandlerMethodArgumentResolverComposite.java
  7. 2
      spring-oxm/src/main/java/org/springframework/oxm/jaxb/Jaxb2Marshaller.java
  8. 2
      spring-web/src/main/java/org/springframework/web/util/HtmlCharacterEntityReferences.java

2
spring-context-support/src/main/java/org/springframework/cache/jcache/interceptor/CacheResolverAdapter.java vendored

@ -46,7 +46,7 @@ class CacheResolverAdapter implements CacheResolver {
} }
CacheInvocationContext<?> cacheInvocationContext = (CacheInvocationContext<?>) context; CacheInvocationContext<?> cacheInvocationContext = (CacheInvocationContext<?>) context;
javax.cache.Cache<Object, Object> cache = target.resolveCache(cacheInvocationContext); javax.cache.Cache<Object, Object> cache = target.resolveCache(cacheInvocationContext);
Assert.notNull(cache, "Cannot resolve cache for '" + context + "' using '" + target + "'"); Assert.notNull(cache, () -> "Cannot resolve cache for '" + context + "' using '" + target + "'");
return Collections.singleton(new JCacheCache(cache)); return Collections.singleton(new JCacheCache(cache));
} }

5
spring-core/src/main/java/org/springframework/core/ParameterizedTypeReference.java

@ -49,8 +49,9 @@ public abstract class ParameterizedTypeReference<T> {
Type type = parameterizedTypeReferenceSubclass.getGenericSuperclass(); Type type = parameterizedTypeReferenceSubclass.getGenericSuperclass();
Assert.isInstanceOf(ParameterizedType.class, type, "Type must be a parameterized type"); Assert.isInstanceOf(ParameterizedType.class, type, "Type must be a parameterized type");
ParameterizedType parameterizedType = (ParameterizedType) type; ParameterizedType parameterizedType = (ParameterizedType) type;
Assert.isTrue(parameterizedType.getActualTypeArguments().length == 1, "Number of type arguments must be 1"); Type[] actualTypeArguments = parameterizedType.getActualTypeArguments();
this.type = parameterizedType.getActualTypeArguments()[0]; Assert.isTrue(actualTypeArguments.length == 1, "Number of type arguments must be 1");
this.type = actualTypeArguments[0];
} }
private ParameterizedTypeReference(Type type) { private ParameterizedTypeReference(Type type) {

2
spring-core/src/main/java/org/springframework/util/xml/TransformerUtils.java

@ -62,7 +62,7 @@ public abstract class TransformerUtils {
*/ */
public static void enableIndenting(Transformer transformer, int indentAmount) { public static void enableIndenting(Transformer transformer, int indentAmount) {
Assert.notNull(transformer, "Transformer must not be null"); Assert.notNull(transformer, "Transformer must not be null");
Assert.isTrue(indentAmount > -1, "The indent amount cannot be less than zero : got " + indentAmount); Assert.isTrue(indentAmount > -1, () -> "The indent amount cannot be less than zero : got " + indentAmount);
transformer.setOutputProperty(OutputKeys.INDENT, "yes"); transformer.setOutputProperty(OutputKeys.INDENT, "yes");
try { try {
// Xalan-specific, but this is the most common XSLT engine in any case // Xalan-specific, but this is the most common XSLT engine in any case

2
spring-messaging/src/main/java/org/springframework/messaging/converter/AbstractMessageConverter.java

@ -139,7 +139,7 @@ public abstract class AbstractMessageConverter implements SmartMessageConverter
*/ */
public void setSerializedPayloadClass(Class<?> payloadClass) { public void setSerializedPayloadClass(Class<?> payloadClass) {
Assert.isTrue(byte[].class == payloadClass || String.class == payloadClass, Assert.isTrue(byte[].class == payloadClass || String.class == payloadClass,
"Payload class must be byte[] or String: " + payloadClass); () -> "Payload class must be byte[] or String: " + payloadClass);
this.serializedPayloadClass = payloadClass; this.serializedPayloadClass = payloadClass;
} }

2
spring-messaging/src/main/java/org/springframework/messaging/handler/annotation/support/HeadersMethodArgumentResolver.java

@ -70,7 +70,7 @@ public class HeadersMethodArgumentResolver implements HandlerMethodArgumentResol
} }
else { else {
Method method = ReflectionUtils.findMethod(paramType, "wrap", Message.class); Method method = ReflectionUtils.findMethod(paramType, "wrap", Message.class);
Assert.notNull(method, "Cannot create accessor of type " + paramType + " for message " + message); Assert.notNull(method, () -> "Cannot create accessor of type " + paramType + " for message " + message);
return ReflectionUtils.invokeMethod(method, null, message); return ReflectionUtils.invokeMethod(method, null, message);
} }
} }

2
spring-messaging/src/main/java/org/springframework/messaging/handler/invocation/HandlerMethodArgumentResolverComposite.java

@ -112,7 +112,7 @@ public class HandlerMethodArgumentResolverComposite implements HandlerMethodArgu
public Object resolveArgument(MethodParameter parameter, Message<?> message) throws Exception { public Object resolveArgument(MethodParameter parameter, Message<?> message) throws Exception {
HandlerMethodArgumentResolver resolver = getArgumentResolver(parameter); HandlerMethodArgumentResolver resolver = getArgumentResolver(parameter);
Assert.notNull(resolver, "Unknown parameter type [" + parameter.getParameterType().getName() + "]"); Assert.notNull(resolver, () -> "Unknown parameter type [" + parameter.getParameterType().getName() + "]");
return resolver.resolveArgument(parameter, message); return resolver.resolveArgument(parameter, message);
} }

2
spring-oxm/src/main/java/org/springframework/oxm/jaxb/Jaxb2Marshaller.java

@ -580,7 +580,7 @@ public class Jaxb2Marshaller implements MimeMarshaller, MimeUnmarshaller, Generi
xmlReader.setFeature("http://xml.org/sax/features/namespace-prefixes", true); xmlReader.setFeature("http://xml.org/sax/features/namespace-prefixes", true);
for (int i = 0; i < resources.length; i++) { for (int i = 0; i < resources.length; i++) {
Assert.notNull(resources[i], "Resource is null"); Assert.notNull(resources[i], "Resource is null");
Assert.isTrue(resources[i].exists(), "Resource " + resources[i] + " does not exist"); Assert.isTrue(resources[i].exists(), () -> "Resource " + resources[i] + " does not exist");
InputSource inputSource = SaxResourceUtils.createInputSource(resources[i]); InputSource inputSource = SaxResourceUtils.createInputSource(resources[i]);
schemaSources[i] = new SAXSource(xmlReader, inputSource); schemaSources[i] = new SAXSource(xmlReader, inputSource);
} }

2
spring-web/src/main/java/org/springframework/web/util/HtmlCharacterEntityReferences.java

@ -89,7 +89,7 @@ class HtmlCharacterEntityReferences {
String key = (String) keys.nextElement(); String key = (String) keys.nextElement();
int referredChar = Integer.parseInt(key); int referredChar = Integer.parseInt(key);
Assert.isTrue((referredChar < 1000 || (referredChar >= 8000 && referredChar < 10000)), Assert.isTrue((referredChar < 1000 || (referredChar >= 8000 && referredChar < 10000)),
"Invalid reference to special HTML entity: " + referredChar); () -> "Invalid reference to special HTML entity: " + referredChar);
int index = (referredChar < 1000 ? referredChar : referredChar - 7000); int index = (referredChar < 1000 ? referredChar : referredChar - 7000);
String reference = entityReferences.getProperty(key); String reference = entityReferences.getProperty(key);
this.characterToEntityReferenceMap[index] = REFERENCE_START + reference + REFERENCE_END; this.characterToEntityReferenceMap[index] = REFERENCE_START + reference + REFERENCE_END;

Loading…
Cancel
Save