Fix "unnecessary @SuppressWarnings" warnings
This commit is contained in:
parent
8d2e125e7b
commit
4c7cafbde6
|
|
@ -38,7 +38,7 @@ import org.springframework.util.ClassUtils;
|
||||||
* @author Juergen Hoeller
|
* @author Juergen Hoeller
|
||||||
* @since 11.11.2003
|
* @since 11.11.2003
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings({ "unchecked", "serial" })
|
@SuppressWarnings({"serial" })
|
||||||
public class DefaultIntroductionAdvisor implements IntroductionAdvisor, ClassFilter, Ordered, Serializable {
|
public class DefaultIntroductionAdvisor implements IntroductionAdvisor, ClassFilter, Ordered, Serializable {
|
||||||
|
|
||||||
private final Advice advice;
|
private final Advice advice;
|
||||||
|
|
|
||||||
|
|
@ -714,7 +714,6 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
|
||||||
return this.resources;
|
return this.resources;
|
||||||
}
|
}
|
||||||
|
|
||||||
@SuppressWarnings("unused")
|
|
||||||
public int size() {
|
public int size() {
|
||||||
return this.resources.size();
|
return this.resources.size();
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -17,12 +17,15 @@
|
||||||
package org.springframework.util;
|
package org.springframework.util;
|
||||||
|
|
||||||
import java.beans.Introspector;
|
import java.beans.Introspector;
|
||||||
|
|
||||||
import java.lang.reflect.Array;
|
import java.lang.reflect.Array;
|
||||||
import java.lang.reflect.Constructor;
|
import java.lang.reflect.Constructor;
|
||||||
import java.lang.reflect.Method;
|
import java.lang.reflect.Method;
|
||||||
import java.lang.reflect.Modifier;
|
import java.lang.reflect.Modifier;
|
||||||
import java.lang.reflect.Proxy;
|
import java.lang.reflect.Proxy;
|
||||||
|
|
||||||
import java.security.AccessControlException;
|
import java.security.AccessControlException;
|
||||||
|
|
||||||
import java.util.Arrays;
|
import java.util.Arrays;
|
||||||
import java.util.Collection;
|
import java.util.Collection;
|
||||||
import java.util.Collections;
|
import java.util.Collections;
|
||||||
|
|
@ -47,7 +50,6 @@ import java.util.Set;
|
||||||
* @see TypeUtils
|
* @see TypeUtils
|
||||||
* @see ReflectionUtils
|
* @see ReflectionUtils
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
public abstract class ClassUtils {
|
public abstract class ClassUtils {
|
||||||
|
|
||||||
/** Suffix for array class names: "[]" */
|
/** Suffix for array class names: "[]" */
|
||||||
|
|
@ -114,9 +116,9 @@ public abstract class ClassUtils {
|
||||||
|
|
||||||
Set<Class<?>> primitiveTypes = new HashSet<Class<?>>(32);
|
Set<Class<?>> primitiveTypes = new HashSet<Class<?>>(32);
|
||||||
primitiveTypes.addAll(primitiveWrapperTypeMap.values());
|
primitiveTypes.addAll(primitiveWrapperTypeMap.values());
|
||||||
primitiveTypes.addAll(Arrays.asList(
|
primitiveTypes.addAll(Arrays.asList(new Class<?>[] {
|
||||||
boolean[].class, byte[].class, char[].class, double[].class,
|
boolean[].class, byte[].class, char[].class, double[].class,
|
||||||
float[].class, int[].class, long[].class, short[].class));
|
float[].class, int[].class, long[].class, short[].class}));
|
||||||
primitiveTypes.add(void.class);
|
primitiveTypes.add(void.class);
|
||||||
for (Class<?> primitiveType : primitiveTypes) {
|
for (Class<?> primitiveType : primitiveTypes) {
|
||||||
primitiveTypeNameMap.put(primitiveType.getName(), primitiveType);
|
primitiveTypeNameMap.put(primitiveType.getName(), primitiveType);
|
||||||
|
|
|
||||||
|
|
@ -64,7 +64,7 @@ public class NullSafeComparator<T> implements Comparator<T> {
|
||||||
* @see #NULLS_LOW
|
* @see #NULLS_LOW
|
||||||
* @see #NULLS_HIGH
|
* @see #NULLS_HIGH
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings({ "unchecked", "rawtypes" })
|
@SuppressWarnings({ "unchecked"})
|
||||||
private NullSafeComparator(boolean nullsLow) {
|
private NullSafeComparator(boolean nullsLow) {
|
||||||
this.nonNullComparator = new ComparableComparator();
|
this.nonNullComparator = new ComparableComparator();
|
||||||
this.nullsLow = nullsLow;
|
this.nullsLow = nullsLow;
|
||||||
|
|
|
||||||
|
|
@ -475,7 +475,6 @@ public class GenericTableMetaDataProvider implements TableMetaDataProvider {
|
||||||
this.type = type;
|
this.type = type;
|
||||||
}
|
}
|
||||||
|
|
||||||
@SuppressWarnings("unused")
|
|
||||||
public String getType() {
|
public String getType() {
|
||||||
return this.type;
|
return this.type;
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -293,7 +293,6 @@ class PersistenceUnitReader {
|
||||||
/**
|
/**
|
||||||
* Parse the {@code property} XML elements.
|
* Parse the {@code property} XML elements.
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
protected void parseProperties(Element persistenceUnit, SpringPersistenceUnitInfo unitInfo) {
|
protected void parseProperties(Element persistenceUnit, SpringPersistenceUnitInfo unitInfo) {
|
||||||
Element propRoot = DomUtils.getChildElementByTagName(persistenceUnit, PROPERTIES);
|
Element propRoot = DomUtils.getChildElementByTagName(persistenceUnit, PROPERTIES);
|
||||||
if (propRoot == null) {
|
if (propRoot == null) {
|
||||||
|
|
@ -310,7 +309,6 @@ class PersistenceUnitReader {
|
||||||
/**
|
/**
|
||||||
* Parse the {@code class} XML elements.
|
* Parse the {@code class} XML elements.
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
protected void parseManagedClasses(Element persistenceUnit, SpringPersistenceUnitInfo unitInfo) {
|
protected void parseManagedClasses(Element persistenceUnit, SpringPersistenceUnitInfo unitInfo) {
|
||||||
List<Element> classes = DomUtils.getChildElementsByTagName(persistenceUnit, MANAGED_CLASS_NAME);
|
List<Element> classes = DomUtils.getChildElementsByTagName(persistenceUnit, MANAGED_CLASS_NAME);
|
||||||
for (Element element : classes) {
|
for (Element element : classes) {
|
||||||
|
|
@ -323,7 +321,6 @@ class PersistenceUnitReader {
|
||||||
/**
|
/**
|
||||||
* Parse the {@code mapping-file} XML elements.
|
* Parse the {@code mapping-file} XML elements.
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
protected void parseMappingFiles(Element persistenceUnit, SpringPersistenceUnitInfo unitInfo) {
|
protected void parseMappingFiles(Element persistenceUnit, SpringPersistenceUnitInfo unitInfo) {
|
||||||
List<Element> files = DomUtils.getChildElementsByTagName(persistenceUnit, MAPPING_FILE_NAME);
|
List<Element> files = DomUtils.getChildElementsByTagName(persistenceUnit, MAPPING_FILE_NAME);
|
||||||
for (Element element : files) {
|
for (Element element : files) {
|
||||||
|
|
@ -337,7 +334,6 @@ class PersistenceUnitReader {
|
||||||
/**
|
/**
|
||||||
* Parse the {@code jar-file} XML elements.
|
* Parse the {@code jar-file} XML elements.
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
protected void parseJarFiles(Element persistenceUnit, SpringPersistenceUnitInfo unitInfo) throws IOException {
|
protected void parseJarFiles(Element persistenceUnit, SpringPersistenceUnitInfo unitInfo) throws IOException {
|
||||||
List<Element> jars = DomUtils.getChildElementsByTagName(persistenceUnit, JAR_FILE_URL);
|
List<Element> jars = DomUtils.getChildElementsByTagName(persistenceUnit, JAR_FILE_URL);
|
||||||
for (Element element : jars) {
|
for (Element element : jars) {
|
||||||
|
|
|
||||||
|
|
@ -126,7 +126,6 @@ public class ServletTestExecutionListener extends AbstractTestExecutionListener
|
||||||
RequestContextHolder.setRequestAttributes(servletWebRequest);
|
RequestContextHolder.setRequestAttributes(servletWebRequest);
|
||||||
|
|
||||||
if (wac instanceof ConfigurableApplicationContext) {
|
if (wac instanceof ConfigurableApplicationContext) {
|
||||||
@SuppressWarnings("resource")
|
|
||||||
ConfigurableApplicationContext configurableApplicationContext = (ConfigurableApplicationContext) wac;
|
ConfigurableApplicationContext configurableApplicationContext = (ConfigurableApplicationContext) wac;
|
||||||
ConfigurableListableBeanFactory bf = configurableApplicationContext.getBeanFactory();
|
ConfigurableListableBeanFactory bf = configurableApplicationContext.getBeanFactory();
|
||||||
bf.registerResolvableDependency(MockHttpServletResponse.class, response);
|
bf.registerResolvableDependency(MockHttpServletResponse.class, response);
|
||||||
|
|
|
||||||
|
|
@ -326,7 +326,6 @@ public abstract class AbstractJpaTests extends AbstractAnnotationAwareTransactio
|
||||||
|
|
||||||
private final LoadTimeWeaver ltw;
|
private final LoadTimeWeaver ltw;
|
||||||
|
|
||||||
@SuppressWarnings("unused")
|
|
||||||
public LoadTimeWeaverInjectingBeanPostProcessor(LoadTimeWeaver ltw) {
|
public LoadTimeWeaverInjectingBeanPostProcessor(LoadTimeWeaver ltw) {
|
||||||
this.ltw = ltw;
|
this.ltw = ltw;
|
||||||
}
|
}
|
||||||
|
|
@ -347,7 +346,6 @@ public abstract class AbstractJpaTests extends AbstractAnnotationAwareTransactio
|
||||||
|
|
||||||
private final ClassLoader shadowingClassLoader;
|
private final ClassLoader shadowingClassLoader;
|
||||||
|
|
||||||
@SuppressWarnings("unused")
|
|
||||||
public ShadowingLoadTimeWeaver(ClassLoader shadowingClassLoader) {
|
public ShadowingLoadTimeWeaver(ClassLoader shadowingClassLoader) {
|
||||||
this.shadowingClassLoader = shadowingClassLoader;
|
this.shadowingClassLoader = shadowingClassLoader;
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -252,7 +252,7 @@ public final class WebAsyncManager {
|
||||||
* @see #getConcurrentResult()
|
* @see #getConcurrentResult()
|
||||||
* @see #getConcurrentResultContext()
|
* @see #getConcurrentResultContext()
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings({ "rawtypes", "unchecked" })
|
@SuppressWarnings({"unchecked" })
|
||||||
public void startCallableProcessing(final Callable<?> callable, Object... processingContext) throws Exception {
|
public void startCallableProcessing(final Callable<?> callable, Object... processingContext) throws Exception {
|
||||||
Assert.notNull(callable, "Callable must not be null");
|
Assert.notNull(callable, "Callable must not be null");
|
||||||
startCallableProcessing(new WebAsyncTask(callable), processingContext);
|
startCallableProcessing(new WebAsyncTask(callable), processingContext);
|
||||||
|
|
|
||||||
|
|
@ -156,7 +156,6 @@ public class ServletContextResourcePatternResolver extends PathMatchingResourceP
|
||||||
* @param result the Set of matching Resources to add to
|
* @param result the Set of matching Resources to add to
|
||||||
* @throws IOException if jar contents could not be retrieved
|
* @throws IOException if jar contents could not be retrieved
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
private void doRetrieveMatchingJarEntries(String jarFilePath, String entryPattern, Set<Resource> result) {
|
private void doRetrieveMatchingJarEntries(String jarFilePath, String entryPattern, Set<Resource> result) {
|
||||||
if (logger.isDebugEnabled()) {
|
if (logger.isDebugEnabled()) {
|
||||||
logger.debug("Searching jar file [" + jarFilePath + "] for entries matching [" + entryPattern + "]");
|
logger.debug("Searching jar file [" + jarFilePath + "] for entries matching [" + entryPattern + "]");
|
||||||
|
|
|
||||||
|
|
@ -103,7 +103,6 @@ public class DefaultMultipartHttpServletRequest extends AbstractMultipartHttpSer
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
public Map<String, String[]> getParameterMap() {
|
public Map<String, String[]> getParameterMap() {
|
||||||
Map<String, String[]> paramMap = new HashMap<String, String[]>();
|
Map<String, String[]> paramMap = new HashMap<String, String[]>();
|
||||||
paramMap.putAll(super.getParameterMap());
|
paramMap.putAll(super.getParameterMap());
|
||||||
|
|
|
||||||
|
|
@ -573,7 +573,6 @@ public abstract class WebUtils {
|
||||||
* @return the value of the parameter, or {@code null}
|
* @return the value of the parameter, or {@code null}
|
||||||
* if the parameter does not exist in given request
|
* if the parameter does not exist in given request
|
||||||
*/
|
*/
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
public static String findParameterValue(ServletRequest request, String name) {
|
public static String findParameterValue(ServletRequest request, String name) {
|
||||||
return findParameterValue(request.getParameterMap(), name);
|
return findParameterValue(request.getParameterMap(), name);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -72,7 +72,6 @@ public class Jaxb2RootElementHttpMessageConverterTest {
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
public void readXmlRootElement() throws Exception {
|
public void readXmlRootElement() throws Exception {
|
||||||
byte[] body = "<rootElement><type s=\"Hello World\"/></rootElement>".getBytes("UTF-8");
|
byte[] body = "<rootElement><type s=\"Hello World\"/></rootElement>".getBytes("UTF-8");
|
||||||
MockHttpInputMessage inputMessage = new MockHttpInputMessage(body);
|
MockHttpInputMessage inputMessage = new MockHttpInputMessage(body);
|
||||||
|
|
@ -81,7 +80,6 @@ public class Jaxb2RootElementHttpMessageConverterTest {
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
public void readXmlRootElementSubclass() throws Exception {
|
public void readXmlRootElementSubclass() throws Exception {
|
||||||
byte[] body = "<rootElement><type s=\"Hello World\"/></rootElement>".getBytes("UTF-8");
|
byte[] body = "<rootElement><type s=\"Hello World\"/></rootElement>".getBytes("UTF-8");
|
||||||
MockHttpInputMessage inputMessage = new MockHttpInputMessage(body);
|
MockHttpInputMessage inputMessage = new MockHttpInputMessage(body);
|
||||||
|
|
@ -90,7 +88,6 @@ public class Jaxb2RootElementHttpMessageConverterTest {
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
public void readXmlType() throws Exception {
|
public void readXmlType() throws Exception {
|
||||||
byte[] body = "<foo s=\"Hello World\"/>".getBytes("UTF-8");
|
byte[] body = "<foo s=\"Hello World\"/>".getBytes("UTF-8");
|
||||||
MockHttpInputMessage inputMessage = new MockHttpInputMessage(body);
|
MockHttpInputMessage inputMessage = new MockHttpInputMessage(body);
|
||||||
|
|
|
||||||
|
|
@ -131,7 +131,6 @@ public class DefaultMultipartActionRequest extends ActionRequestWrapper implemen
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
@SuppressWarnings("unchecked")
|
|
||||||
public Map<String, String[]> getParameterMap() {
|
public Map<String, String[]> getParameterMap() {
|
||||||
Map<String, String[]> paramMap = new HashMap<String, String[]>();
|
Map<String, String[]> paramMap = new HashMap<String, String[]>();
|
||||||
paramMap.putAll(super.getParameterMap());
|
paramMap.putAll(super.getParameterMap());
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue