Author: ggregory
Date: Wed Oct 15 20:15:17 2014
New Revision: 1632171
URL: http://svn.apache.org/r1632171
Log:
Add final modifier to method parameters.
Add final modifier to local variables.
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BaseDynaBeanMapDecorator.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaClass.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanAccessLanguageException.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanComparator.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanIntrospectionData.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanMap.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPredicate.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPropertyValueChangeClosure.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPropertyValueEqualsPredicate.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanToPropertyValueTransformer.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanUtils.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanUtilsBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanUtilsBean2.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ConstructorUtils.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ContextClassLoaderLocal.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ConversionException.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ConvertUtils.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ConvertUtilsBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ConvertUtilsBean2.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ConvertingWrapDynaBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/DefaultBeanIntrospector.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/DefaultIntrospectionContext.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/DynaBeanMapDecorator.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/DynaBeanPropertyMapDecorator.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/DynaProperty.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/FluentPropertyBeanIntrospector.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/JDBCDynaClass.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/LazyDynaBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/LazyDynaClass.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/LazyDynaList.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/LazyDynaMap.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/MappedPropertyDescriptor.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/MethodUtils.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/NestedNullException.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/PropertyUtils.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/PropertyUtilsBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ResultSetDynaClass.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/ResultSetIterator.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/RowSetDynaClass.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/SuppressPropertiesBeanIntrospector.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/WeakFastHashMap.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/WrapDynaBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/WrapDynaClass.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/AbstractArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/AbstractConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/ArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/BigDecimalConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/BigIntegerConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/BooleanArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/BooleanConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/ByteArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/ByteConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/CalendarConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/CharacterArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/CharacterConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/ClassConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/ConverterFacade.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/DateConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/DateTimeConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/DoubleArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/DoubleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/FileConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/FloatArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/FloatConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/IntegerArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/IntegerConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/LongArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/LongConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/NumberConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/ShortArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/ShortConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/SqlDateConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/SqlTimeConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/SqlTimestampConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/StringArrayConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/StringConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/converters/URLConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/expression/DefaultResolver.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/BaseLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/LocaleBeanUtils.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/LocaleBeanUtilsBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/LocaleConvertUtils.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/LocaleConvertUtilsBean.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/BigDecimalLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/BigIntegerLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/ByteLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/DateLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/DecimalLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/DoubleLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/FloatLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/IntegerLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/LongLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/ShortLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/SqlDateLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/SqlTimeLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/SqlTimestampLocaleConverter.java
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/locale/converters/StringLocaleConverter.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/A.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/AbstractChild.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/AbstractParent.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/AlphaBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BasicDynaBeanTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanComparatorTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanIntrospectionDataTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanMapTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanPredicateTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanPropertyValueChangeClosureTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanPropertyValueEqualsPredicateTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanToPropertyValueTransformerTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanUtils2TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanUtilsBenchCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanWithInnerBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BeanificationTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BenchBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/BetaBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/ConstructorUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/ConvertUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/DefaultIntrospectionContextTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/DynaBeanMapDecoratorTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/DynaBeanUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/DynaPropertyTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/DynaPropertyUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/DynaResultSetTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/DynaRowSetTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/ExtendMapBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/FluentIntrospectionTestBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/FluentPropertyBeanIntrospectorTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/IndexedPropertyTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/IndexedTestBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/LazyDynaBeanTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/LazyDynaClassTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/LazyDynaListTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/LazyDynaMapTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/MappedPropertyTestBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/MappedPropertyTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/MethodUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/NestedTestBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/PrimitiveBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/PropertyUtilsBenchCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/PropertyUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/PropsFirstPropertyUtilsBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/SonOfAlphaBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/SuppressPropertiesBeanIntrospectorTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/TestBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/TestResultSet.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/TestResultSetMetaData.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/ThrowExceptionConverter.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/WrapDynaBeanTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira157TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira18TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira273TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira298TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira339TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira345TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira347TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira349TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira357TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira358TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira359TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira368TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira369TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira381TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira411TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira422TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira454TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira456TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira458TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira463TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira465TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira61TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira87TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/Jira92TestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/other/Jira18BeanFactory.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/other/Jira298BeanFactory.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/other/Jira61BeanFactory.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/bugs/other/Jira87BeanFactory.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/ArrayConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/BigDecimalConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/BigIntegerConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/BooleanArrayConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/BooleanConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/ByteConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/CalendarConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/CharacterConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/ClassConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/ClassReloader.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/ClassReloaderTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/ConverterTestSuite.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/DateConverterTestBase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/DateConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/DoubleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/FileConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/FloatConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/IntegerConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/LongConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/MemoryTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/NumberConverterTestBase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/ShortConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/SqlDateConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/SqlTimeConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/SqlTimestampConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/StringArrayConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/StringConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/converters/URLConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/expression/DefaultResolverTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/LocaleBeanUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/LocaleBeanificationTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/LocaleConvertTestSuite.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/LocaleConvertUtilsTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/BaseLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/BigDecimalLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/BigIntegerLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/ByteLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/DateLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/DoubleLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/FloatLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/IntegerLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/LongLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/locale/converters/ShortLocaleConverterTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/memoryleaktests/MemoryLeakTestCase.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/memoryleaktests/pojotests/CustomInteger.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/memoryleaktests/pojotests/SomeMappedPojo.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/memoryleaktests/pojotests/SomePojo.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/priv/PackageBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/priv/PrivateBean.java
commons/proper/beanutils/trunk/src/test/java/org/apache/commons/beanutils/priv/PublicSubBean.java
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BaseDynaBeanMapDecorator.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BaseDynaBeanMapDecorator.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BaseDynaBeanMapDecorator.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BaseDynaBeanMapDecorator.java
Wed Oct 15 20:15:17 2014
@@ -64,7 +64,7 @@ public abstract class BaseDynaBeanMapDec
* @param dynaBean The dyna bean being decorated
* @throws IllegalArgumentException if the {@link DynaBean} is null.
*/
- public BaseDynaBeanMapDecorator(DynaBean dynaBean) {
+ public BaseDynaBeanMapDecorator(final DynaBean dynaBean) {
this(dynaBean, true);
}
@@ -76,7 +76,7 @@ public abstract class BaseDynaBeanMapDec
* otherwise <code>false</code>
* @throws IllegalArgumentException if the {@link DynaBean} is null.
*/
- public BaseDynaBeanMapDecorator(DynaBean dynaBean, boolean readOnly) {
+ public BaseDynaBeanMapDecorator(final DynaBean dynaBean, final boolean
readOnly) {
if (dynaBean == null) {
throw new IllegalArgumentException("DynaBean is null");
}
@@ -117,9 +117,9 @@ public abstract class BaseDynaBeanMapDec
* @return <code>true</code> if one of the {@link DynaBean}'s
* properties contains a specified value.
*/
- public boolean containsKey(Object key) {
- DynaClass dynaClass = getDynaBean().getDynaClass();
- DynaProperty dynaProperty = dynaClass.getDynaProperty(toString(key));
+ public boolean containsKey(final Object key) {
+ final DynaClass dynaClass = getDynaBean().getDynaClass();
+ final DynaProperty dynaProperty =
dynaClass.getDynaProperty(toString(key));
return (dynaProperty == null ? false : true);
}
@@ -132,11 +132,11 @@ public abstract class BaseDynaBeanMapDec
* properties contains the specified value, otherwise
* <code>false</code>.
*/
- public boolean containsValue(Object value) {
- DynaProperty[] properties = getDynaProperties();
+ public boolean containsValue(final Object value) {
+ final DynaProperty[] properties = getDynaProperties();
for (int i = 0; i < properties.length; i++) {
- String key = properties[i].getName();
- Object prop = getDynaBean().get(key);
+ final String key = properties[i].getName();
+ final Object prop = getDynaBean().get(key);
if (value == null) {
if (prop == null) {
return true;
@@ -161,11 +161,11 @@ public abstract class BaseDynaBeanMapDec
* property name/value pairs
*/
public Set<Map.Entry<K, Object>> entrySet() {
- DynaProperty[] properties = getDynaProperties();
- Set<Map.Entry<K, Object>> set = new HashSet<Map.Entry<K,
Object>>(properties.length);
+ final DynaProperty[] properties = getDynaProperties();
+ final Set<Map.Entry<K, Object>> set = new HashSet<Map.Entry<K,
Object>>(properties.length);
for (int i = 0; i < properties.length; i++) {
- K key = convertKey(properties[i].getName());
- Object value = getDynaBean().get(properties[i].getName());
+ final K key = convertKey(properties[i].getName());
+ final Object value = getDynaBean().get(properties[i].getName());
set.add(new MapEntry<K>(key, value));
}
return Collections.unmodifiableSet(set);
@@ -178,7 +178,7 @@ public abstract class BaseDynaBeanMapDec
* @param key The {@link DynaBean}'s property name
* @return The value for the specified property.
*/
- public Object get(Object key) {
+ public Object get(final Object key) {
return getDynaBean().get(toString(key));
}
@@ -210,7 +210,7 @@ public abstract class BaseDynaBeanMapDec
}
// Create a Set of the keys
- DynaProperty[] properties = getDynaProperties();
+ final DynaProperty[] properties = getDynaProperties();
Set<K> set = new HashSet<K>(properties.length);
for (int i = 0; i < properties.length; i++) {
set.add(convertKey(properties[i].getName()));
@@ -218,7 +218,7 @@ public abstract class BaseDynaBeanMapDec
set = Collections.unmodifiableSet(set);
// Cache the keySet if Not a MutableDynaClass
- DynaClass dynaClass = getDynaBean().getDynaClass();
+ final DynaClass dynaClass = getDynaBean().getDynaClass();
if (!(dynaClass instanceof MutableDynaClass)) {
keySet = set;
}
@@ -237,12 +237,12 @@ public abstract class BaseDynaBeanMapDec
* @throws UnsupportedOperationException if
* <code>isReadOnly()</code> is true.
*/
- public Object put(K key, Object value) {
+ public Object put(final K key, final Object value) {
if (isReadOnly()) {
throw new UnsupportedOperationException("Map is read only");
}
- String property = toString(key);
- Object previous = getDynaBean().get(property);
+ final String property = toString(key);
+ final Object previous = getDynaBean().get(property);
getDynaBean().set(property, value);
return previous;
}
@@ -254,11 +254,11 @@ public abstract class BaseDynaBeanMapDec
* @throws UnsupportedOperationException if
* <code>isReadOnly()</code> is true.
*/
- public void putAll(Map<? extends K, ? extends Object> map) {
+ public void putAll(final Map<? extends K, ? extends Object> map) {
if (isReadOnly()) {
throw new UnsupportedOperationException("Map is read only");
}
- for (Map.Entry<? extends K, ?> e : map.entrySet()) {
+ for (final Map.Entry<? extends K, ?> e : map.entrySet()) {
put(e.getKey(), e.getValue());
}
}
@@ -270,7 +270,7 @@ public abstract class BaseDynaBeanMapDec
* @return the value removed
* @throws UnsupportedOperationException
*/
- public Object remove(Object key) {
+ public Object remove(final Object key) {
throw new UnsupportedOperationException();
}
@@ -290,11 +290,11 @@ public abstract class BaseDynaBeanMapDec
* @return Unmodifiable collection of values.
*/
public Collection<Object> values() {
- DynaProperty[] properties = getDynaProperties();
- List<Object> values = new ArrayList<Object>(properties.length);
+ final DynaProperty[] properties = getDynaProperties();
+ final List<Object> values = new ArrayList<Object>(properties.length);
for (int i = 0; i < properties.length; i++) {
- String key = properties[i].getName();
- Object value = getDynaBean().get(key);
+ final String key = properties[i].getName();
+ final Object value = getDynaBean().get(key);
values.add(value);
}
return Collections.unmodifiableList(values);
@@ -339,7 +339,7 @@ public abstract class BaseDynaBeanMapDec
* @param obj The Object to convert
* @return String representation of the object
*/
- private String toString(Object obj) {
+ private String toString(final Object obj) {
return (obj == null ? null : obj.toString());
}
@@ -349,16 +349,16 @@ public abstract class BaseDynaBeanMapDec
private static class MapEntry<K> implements Map.Entry<K, Object> {
private final K key;
private final Object value;
- MapEntry(K key, Object value) {
+ MapEntry(final K key, final Object value) {
this.key = key;
this.value = value;
}
@Override
- public boolean equals(Object o) {
+ public boolean equals(final Object o) {
if (!(o instanceof Map.Entry)) {
return false;
}
- Map.Entry<?, ?> e = (Map.Entry<?, ?>)o;
+ final Map.Entry<?, ?> e = (Map.Entry<?, ?>)o;
return ((key.equals(e.getKey())) &&
(value == null ? e.getValue() == null
: value.equals(e.getValue())));
@@ -373,7 +373,7 @@ public abstract class BaseDynaBeanMapDec
public Object getValue() {
return value;
}
- public Object setValue(Object value) {
+ public Object setValue(final Object value) {
throw new UnsupportedOperationException();
}
}
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaBean.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaBean.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaBean.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaBean.java
Wed Oct 15 20:15:17 2014
@@ -52,7 +52,7 @@ public class BasicDynaBean implements Dy
*
* @param dynaClass The DynaClass we are associated with
*/
- public BasicDynaBean(DynaClass dynaClass) {
+ public BasicDynaBean(final DynaClass dynaClass) {
super();
this.dynaClass = dynaClass;
@@ -113,9 +113,9 @@ public class BasicDynaBean implements Dy
* @exception IllegalArgumentException if there is no property
* of the specified name
*/
- public boolean contains(String name, String key) {
+ public boolean contains(final String name, final String key) {
- Object value = values.get(name);
+ final Object value = values.get(name);
if (value == null) {
throw new NullPointerException
("No mapped value for '" + name + "(" + key + ")'");
@@ -138,16 +138,16 @@ public class BasicDynaBean implements Dy
* @exception IllegalArgumentException if there is no property
* of the specified name
*/
- public Object get(String name) {
+ public Object get(final String name) {
// Return any non-null value for the specified property
- Object value = values.get(name);
+ final Object value = values.get(name);
if (value != null) {
return (value);
}
// Return a null value for a non-primitive property
- Class<?> type = getDynaProperty(name).getType();
+ final Class<?> type = getDynaProperty(name).getType();
if (!type.isPrimitive()) {
return (value);
}
@@ -192,9 +192,9 @@ public class BasicDynaBean implements Dy
* @exception NullPointerException if no array or List has been
* initialized for this property
*/
- public Object get(String name, int index) {
+ public Object get(final String name, final int index) {
- Object value = values.get(name);
+ final Object value = values.get(name);
if (value == null) {
throw new NullPointerException
("No indexed value for '" + name + "[" + index + "]'");
@@ -223,9 +223,9 @@ public class BasicDynaBean implements Dy
* @exception IllegalArgumentException if the specified property
* exists, but is not mapped
*/
- public Object get(String name, String key) {
+ public Object get(final String name, final String key) {
- Object value = values.get(name);
+ final Object value = values.get(name);
if (value == null) {
throw new NullPointerException
("No mapped value for '" + name + "(" + key + ")'");
@@ -263,9 +263,9 @@ public class BasicDynaBean implements Dy
* @exception IllegalArgumentException if there is no property
* of the specified name
*/
- public void remove(String name, String key) {
+ public void remove(final String name, final String key) {
- Object value = values.get(name);
+ final Object value = values.get(name);
if (value == null) {
throw new NullPointerException
("No mapped value for '" + name + "(" + key + ")'");
@@ -292,9 +292,9 @@ public class BasicDynaBean implements Dy
* @exception NullPointerException if an attempt is made to set a
* primitive property to null
*/
- public void set(String name, Object value) {
+ public void set(final String name, final Object value) {
- DynaProperty descriptor = getDynaProperty(name);
+ final DynaProperty descriptor = getDynaProperty(name);
if (value == null) {
if (descriptor.getType().isPrimitive()) {
throw new NullPointerException
@@ -328,9 +328,9 @@ public class BasicDynaBean implements Dy
* @exception IndexOutOfBoundsException if the specified index
* is outside the range of the underlying property
*/
- public void set(String name, int index, Object value) {
+ public void set(final String name, final int index, final Object value) {
- Object prop = values.get(name);
+ final Object prop = values.get(name);
if (prop == null) {
throw new NullPointerException
("No indexed value for '" + name + "[" + index + "]'");
@@ -339,11 +339,12 @@ public class BasicDynaBean implements Dy
} else if (prop instanceof List) {
try {
@SuppressWarnings("unchecked")
+ final
// This is safe to cast because list properties are always
// of type Object
List<Object> list = (List<Object>) prop;
list.set(index, value);
- } catch (ClassCastException e) {
+ } catch (final ClassCastException e) {
throw new ConversionException(e.getMessage());
}
} else {
@@ -368,14 +369,15 @@ public class BasicDynaBean implements Dy
* @exception IllegalArgumentException if the specified property
* exists, but is not mapped
*/
- public void set(String name, String key, Object value) {
+ public void set(final String name, final String key, final Object value) {
- Object prop = values.get(name);
+ final Object prop = values.get(name);
if (prop == null) {
throw new NullPointerException
("No mapped value for '" + name + "(" + key + ")'");
} else if (prop instanceof Map) {
@SuppressWarnings("unchecked")
+ final
// This is safe to cast because mapped properties are always
// maps of types String -> Object
Map<String, Object> map = (Map<String, Object>) prop;
@@ -400,9 +402,9 @@ public class BasicDynaBean implements Dy
* @exception IllegalArgumentException if this is not a valid property
* name for our DynaClass
*/
- protected DynaProperty getDynaProperty(String name) {
+ protected DynaProperty getDynaProperty(final String name) {
- DynaProperty descriptor = getDynaClass().getDynaProperty(name);
+ final DynaProperty descriptor = getDynaClass().getDynaProperty(name);
if (descriptor == null) {
throw new IllegalArgumentException
("Invalid property name '" + name + "'");
@@ -420,7 +422,7 @@ public class BasicDynaBean implements Dy
* @return <code>true</code> if the source class is assignable to the
* destination class, otherwise <code>false</code>
*/
- protected boolean isAssignable(Class<?> dest, Class<?> source) {
+ protected boolean isAssignable(final Class<?> dest, final Class<?> source)
{
if (dest.isAssignableFrom(source) ||
((dest == Boolean.TYPE) && (source == Boolean.class)) ||
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaClass.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaClass.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaClass.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BasicDynaClass.java
Wed Oct 15 20:15:17 2014
@@ -58,7 +58,7 @@ public class BasicDynaClass implements D
* @param name Name of this DynaBean class
* @param dynaBeanClass The implementation class for new instances
*/
- public BasicDynaClass(String name, Class<?> dynaBeanClass) {
+ public BasicDynaClass(final String name, final Class<?> dynaBeanClass) {
this(name, dynaBeanClass, null);
@@ -72,8 +72,8 @@ public class BasicDynaClass implements D
* @param dynaBeanClass The implementation class for new intances
* @param properties Property descriptors for the supported properties
*/
- public BasicDynaClass(String name, Class<?> dynaBeanClass,
- DynaProperty[] properties) {
+ public BasicDynaClass(final String name, Class<?> dynaBeanClass,
+ final DynaProperty[] properties) {
super();
if (name != null) {
@@ -169,7 +169,7 @@ public class BasicDynaClass implements D
*
* @exception IllegalArgumentException if no property name is specified
*/
- public DynaProperty getDynaProperty(String name) {
+ public DynaProperty getDynaProperty(final String name) {
if (name == null) {
throw new IllegalArgumentException
@@ -219,7 +219,7 @@ public class BasicDynaClass implements D
}
// Invoke the constructor to create a new bean instance
return ((DynaBean) constructor.newInstance(constructorValues));
- } catch (InvocationTargetException e) {
+ } catch (final InvocationTargetException e) {
throw new InstantiationException
(e.getTargetException().getMessage());
}
@@ -257,7 +257,7 @@ public class BasicDynaClass implements D
* @exception IllegalArgumentException if the specified Class does not
* implement the <code>DynaBean</code> interface
*/
- protected void setDynaBeanClass(Class<?> dynaBeanClass) {
+ protected void setDynaBeanClass(final Class<?> dynaBeanClass) {
// Validate the argument type specified
if (dynaBeanClass.isInterface()) {
@@ -274,7 +274,7 @@ public class BasicDynaClass implements D
// Identify the Constructor we will use in newInstance()
try {
this.constructor = dynaBeanClass.getConstructor(constructorTypes);
- } catch (NoSuchMethodException e) {
+ } catch (final NoSuchMethodException e) {
throw new IllegalArgumentException
("Class " + dynaBeanClass.getName() +
" does not have an appropriate constructor");
@@ -289,7 +289,7 @@ public class BasicDynaClass implements D
*
* @param properties List of dynamic properties to be supported
*/
- protected void setProperties(DynaProperty[] properties) {
+ protected void setProperties(final DynaProperty[] properties) {
this.properties = properties;
propertiesMap.clear();
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanAccessLanguageException.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanAccessLanguageException.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanAccessLanguageException.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanAccessLanguageException.java
Wed Oct 15 20:15:17 2014
@@ -42,7 +42,7 @@ public class BeanAccessLanguageException
*
* @param message the detail message explaining this exception
*/
- public BeanAccessLanguageException(String message) {
+ public BeanAccessLanguageException(final String message) {
super(message);
}
}
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanComparator.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanComparator.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanComparator.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanComparator.java
Wed Oct 15 20:15:17 2014
@@ -83,7 +83,7 @@ public class BeanComparator<T> implement
* property. See {@link PropertyUtilsBean} for property query language
syntax.
* If the property passed in is null then the actual objects will be
compared
*/
- public BeanComparator( String property ) {
+ public BeanComparator( final String property ) {
this( property, ComparableComparator.getInstance() );
}
@@ -103,7 +103,7 @@ public class BeanComparator<T> implement
* contains null values, a suitable comparator
* may be supplied in this constructor.
*/
- public BeanComparator( String property, Comparator<?> comparator ) {
+ public BeanComparator( final String property, final Comparator<?>
comparator ) {
setProperty( property );
if (comparator != null) {
this.comparator = comparator;
@@ -118,7 +118,7 @@ public class BeanComparator<T> implement
* @param property String method name to call to compare
* If the property passed in is null then the actual objects will be
compared
*/
- public void setProperty( String property ) {
+ public void setProperty( final String property ) {
this.property = property;
}
@@ -152,7 +152,7 @@ public class BeanComparator<T> implement
* @param o2 Object The second bean to get data from to compare
* @return int negative or positive based on order
*/
- public int compare( T o1, T o2 ) {
+ public int compare( final T o1, final T o2 ) {
if ( property == null ) {
// compare the actual objects
@@ -160,17 +160,17 @@ public class BeanComparator<T> implement
}
try {
- Object value1 = PropertyUtils.getProperty( o1, property );
- Object value2 = PropertyUtils.getProperty( o2, property );
+ final Object value1 = PropertyUtils.getProperty( o1, property );
+ final Object value2 = PropertyUtils.getProperty( o2, property );
return internalCompare( value1, value2 );
}
- catch ( IllegalAccessException iae ) {
+ catch ( final IllegalAccessException iae ) {
throw new RuntimeException( "IllegalAccessException: " +
iae.toString() );
}
- catch ( InvocationTargetException ite ) {
+ catch ( final InvocationTargetException ite ) {
throw new RuntimeException( "InvocationTargetException: " +
ite.toString() );
}
- catch ( NoSuchMethodException nsme ) {
+ catch ( final NoSuchMethodException nsme ) {
throw new RuntimeException( "NoSuchMethodException: " +
nsme.toString() );
}
}
@@ -183,7 +183,7 @@ public class BeanComparator<T> implement
* @return whether the the comparators are equal or not
*/
@Override
- public boolean equals(Object o) {
+ public boolean equals(final Object o) {
if (this == o) {
return true;
}
@@ -230,8 +230,9 @@ public class BeanComparator<T> implement
* @param val2 the second value to be compared
* @return the result of the comparison
*/
- private int internalCompare(Object val1, Object val2) {
+ private int internalCompare(final Object val1, final Object val2) {
@SuppressWarnings("rawtypes")
+ final
// to make the compiler happy
Comparator c = comparator;
return c.compare(val1, val2);
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanIntrospectionData.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanIntrospectionData.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanIntrospectionData.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanIntrospectionData.java
Wed Oct 15 20:15:17 2014
@@ -55,7 +55,7 @@ class BeanIntrospectionData {
*
* @param descs the array with the descriptors of the available properties
*/
- public BeanIntrospectionData(PropertyDescriptor[] descs) {
+ public BeanIntrospectionData(final PropertyDescriptor[] descs) {
this(descs, setUpWriteMethodNames(descs));
}
@@ -66,7 +66,7 @@ class BeanIntrospectionData {
* @param descs the array with the descriptors of the available properties
* @param writeMethNames the map with the names of write methods
*/
- BeanIntrospectionData(PropertyDescriptor[] descs, Map<String, String>
writeMethNames) {
+ BeanIntrospectionData(final PropertyDescriptor[] descs, final Map<String,
String> writeMethNames) {
descriptors = descs;
writeMethodNames = writeMethNames;
}
@@ -87,8 +87,8 @@ class BeanIntrospectionData {
* @param name the name of the property in question
* @return the {@code PropertyDescriptor} for this property or <b>null</b>
*/
- public PropertyDescriptor getDescriptor(String name) {
- for (PropertyDescriptor pd : getDescriptors()) {
+ public PropertyDescriptor getDescriptor(final String name) {
+ for (final PropertyDescriptor pd : getDescriptors()) {
if (name.equals(pd.getName())) {
return pd;
}
@@ -109,17 +109,17 @@ class BeanIntrospectionData {
* @param desc the {@code PropertyDescriptor} of the desired property
* @return the write method for this property or <b>null</b> if there is
none
*/
- public Method getWriteMethod(Class<?> beanCls, PropertyDescriptor desc) {
+ public Method getWriteMethod(final Class<?> beanCls, final
PropertyDescriptor desc) {
Method method = desc.getWriteMethod();
if (method == null) {
- String methodName = writeMethodNames.get(desc.getName());
+ final String methodName = writeMethodNames.get(desc.getName());
if (methodName != null) {
method = MethodUtils.getAccessibleMethod(beanCls, methodName,
desc.getPropertyType());
if (method != null) {
try {
desc.setWriteMethod(method);
- } catch (IntrospectionException e) {
+ } catch (final IntrospectionException e) {
// ignore, in this case the method is not cached
}
}
@@ -138,10 +138,10 @@ class BeanIntrospectionData {
* @param descs the array with the descriptors of the available properties
* @return the map with the names of write methods for properties
*/
- private static Map<String, String>
setUpWriteMethodNames(PropertyDescriptor[] descs) {
- Map<String, String> methods = new HashMap<String, String>();
- for (PropertyDescriptor pd : descs) {
- Method method = pd.getWriteMethod();
+ private static Map<String, String> setUpWriteMethodNames(final
PropertyDescriptor[] descs) {
+ final Map<String, String> methods = new HashMap<String, String>();
+ for (final PropertyDescriptor pd : descs) {
+ final Method method = pd.getWriteMethod();
if (method != null) {
methods.put(pd.getName(), method.getName());
}
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanMap.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanMap.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanMap.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanMap.java
Wed Oct 15 20:15:17 2014
@@ -81,11 +81,11 @@ public class BeanMap extends AbstractMap
throw new UnsupportedOperationException();
}
@Override
- public boolean containsKey(Object key) {
+ public boolean containsKey(final Object key) {
return typeTransformers.containsKey(key);
}
@Override
- public boolean containsValue(Object value) {
+ public boolean containsValue(final Object value) {
return typeTransformers.containsValue(value);
}
@Override
@@ -93,7 +93,7 @@ public class BeanMap extends AbstractMap
return typeTransformers.entrySet();
}
@Override
- public Object get(Object key) {
+ public Object get(final Object key) {
return typeTransformers.get(key);
}
@Override
@@ -105,15 +105,15 @@ public class BeanMap extends AbstractMap
return typeTransformers.keySet();
}
@Override
- public Object put(Object key, Object value) {
+ public Object put(final Object key, final Object value) {
throw new UnsupportedOperationException();
}
@Override
- public void putAll(Map m) {
+ public void putAll(final Map m) {
throw new UnsupportedOperationException();
}
@Override
- public Object remove(Object key) {
+ public Object remove(final Object key) {
throw new UnsupportedOperationException();
}
@Override
@@ -127,12 +127,12 @@ public class BeanMap extends AbstractMap
};
private static Map<Class<? extends Object>, Transformer>
createTypeTransformers() {
- Map<Class<? extends Object>, Transformer> defaultTransformers =
+ final Map<Class<? extends Object>, Transformer> defaultTransformers =
new HashMap<Class<? extends Object>, Transformer>();
defaultTransformers.put(
Boolean.TYPE,
new Transformer() {
- public Object transform( Object input ) {
+ public Object transform( final Object input ) {
return Boolean.valueOf( input.toString() );
}
}
@@ -140,7 +140,7 @@ public class BeanMap extends AbstractMap
defaultTransformers.put(
Character.TYPE,
new Transformer() {
- public Object transform( Object input ) {
+ public Object transform( final Object input ) {
return new Character( input.toString().charAt( 0 ) );
}
}
@@ -148,7 +148,7 @@ public class BeanMap extends AbstractMap
defaultTransformers.put(
Byte.TYPE,
new Transformer() {
- public Object transform( Object input ) {
+ public Object transform( final Object input ) {
return Byte.valueOf( input.toString() );
}
}
@@ -156,7 +156,7 @@ public class BeanMap extends AbstractMap
defaultTransformers.put(
Short.TYPE,
new Transformer() {
- public Object transform( Object input ) {
+ public Object transform( final Object input ) {
return Short.valueOf( input.toString() );
}
}
@@ -164,7 +164,7 @@ public class BeanMap extends AbstractMap
defaultTransformers.put(
Integer.TYPE,
new Transformer() {
- public Object transform( Object input ) {
+ public Object transform( final Object input ) {
return Integer.valueOf( input.toString() );
}
}
@@ -172,7 +172,7 @@ public class BeanMap extends AbstractMap
defaultTransformers.put(
Long.TYPE,
new Transformer() {
- public Object transform( Object input ) {
+ public Object transform( final Object input ) {
return Long.valueOf( input.toString() );
}
}
@@ -180,7 +180,7 @@ public class BeanMap extends AbstractMap
defaultTransformers.put(
Float.TYPE,
new Transformer() {
- public Object transform( Object input ) {
+ public Object transform( final Object input ) {
return Float.valueOf( input.toString() );
}
}
@@ -188,7 +188,7 @@ public class BeanMap extends AbstractMap
defaultTransformers.put(
Double.TYPE,
new Transformer() {
- public Object transform( Object input ) {
+ public Object transform( final Object input ) {
return Double.valueOf( input.toString() );
}
}
@@ -213,7 +213,7 @@ public class BeanMap extends AbstractMap
*
* @param bean the bean for this map to operate on
*/
- public BeanMap(Object bean) {
+ public BeanMap(final Object bean) {
this.bean = bean;
initialise();
}
@@ -259,7 +259,7 @@ public class BeanMap extends AbstractMap
*/
@Override
public Object clone() throws CloneNotSupportedException {
- BeanMap newMap = (BeanMap)super.clone();
+ final BeanMap newMap = (BeanMap)super.clone();
if(bean == null) {
// no bean, just an empty bean map at the moment. return a newly
@@ -268,12 +268,12 @@ public class BeanMap extends AbstractMap
}
Object newBean = null;
- Class<? extends Object> beanClass = bean.getClass(); // Cannot throw
Exception
+ final Class<? extends Object> beanClass = bean.getClass(); // Cannot
throw Exception
try {
newBean = beanClass.newInstance();
- } catch (Exception e) {
+ } catch (final Exception e) {
// unable to instantiate
- CloneNotSupportedException cnse = new CloneNotSupportedException
+ final CloneNotSupportedException cnse = new
CloneNotSupportedException
("Unable to instantiate the underlying bean \"" +
beanClass.getName() + "\": " + e);
BeanUtils.initCause(cnse, e);
@@ -282,8 +282,8 @@ public class BeanMap extends AbstractMap
try {
newMap.setBean(newBean);
- } catch (Exception exception) {
- CloneNotSupportedException cnse = new CloneNotSupportedException
+ } catch (final Exception exception) {
+ final CloneNotSupportedException cnse = new
CloneNotSupportedException
("Unable to set bean in the cloned bean map: " +
exception);
BeanUtils.initCause(cnse, exception);
@@ -294,15 +294,15 @@ public class BeanMap extends AbstractMap
// copy only properties that are readable and writable. If its
// not readable, we can't get the value from the old map. If
// its not writable, we can't write a value into the new map.
- Iterator<?> readableKeys = readMethods.keySet().iterator();
+ final Iterator<?> readableKeys = readMethods.keySet().iterator();
while(readableKeys.hasNext()) {
- Object key = readableKeys.next();
+ final Object key = readableKeys.next();
if(getWriteMethod(key) != null) {
newMap.put(key, get(key));
}
}
- } catch (Exception exception) {
- CloneNotSupportedException cnse = new CloneNotSupportedException
+ } catch (final Exception exception) {
+ final CloneNotSupportedException cnse = new
CloneNotSupportedException
("Unable to copy bean values to cloned bean map: " +
exception);
BeanUtils.initCause(cnse, exception);
@@ -318,10 +318,10 @@ public class BeanMap extends AbstractMap
*
* @param map the BeanMap whose properties to put
*/
- public void putAllWriteable(BeanMap map) {
- Iterator<?> readableKeys = map.readMethods.keySet().iterator();
+ public void putAllWriteable(final BeanMap map) {
+ final Iterator<?> readableKeys = map.readMethods.keySet().iterator();
while (readableKeys.hasNext()) {
- Object key = readableKeys.next();
+ final Object key = readableKeys.next();
if (getWriteMethod(key) != null) {
this.put(key, map.get(key));
}
@@ -348,8 +348,8 @@ public class BeanMap extends AbstractMap
beanClass = bean.getClass();
bean = beanClass.newInstance();
}
- catch (Exception e) {
- UnsupportedOperationException uoe =
+ catch (final Exception e) {
+ final UnsupportedOperationException uoe =
new UnsupportedOperationException("Could not create new
instance of class: " + beanClass);
BeanUtils.initCause(uoe, e);
throw uoe;
@@ -372,8 +372,8 @@ public class BeanMap extends AbstractMap
* true if the bean does define a property with that name
*/
@Override
- public boolean containsKey(Object name) {
- Method method = getReadMethod(name);
+ public boolean containsKey(final Object name) {
+ final Method method = getReadMethod(name);
return method != null;
}
@@ -386,7 +386,7 @@ public class BeanMap extends AbstractMap
* current value is that object, false otherwise
*/
@Override
- public boolean containsValue(Object value) {
+ public boolean containsValue(final Object value) {
// use default implementation
return super.containsValue(value);
}
@@ -407,23 +407,23 @@ public class BeanMap extends AbstractMap
* @return the value of the property with that name
*/
@Override
- public Object get(Object name) {
+ public Object get(final Object name) {
if ( bean != null ) {
- Method method = getReadMethod( name );
+ final Method method = getReadMethod( name );
if ( method != null ) {
try {
return method.invoke( bean, NULL_ARGUMENTS );
}
- catch ( IllegalAccessException e ) {
+ catch ( final IllegalAccessException e ) {
logWarn( e );
}
- catch ( IllegalArgumentException e ) {
+ catch ( final IllegalArgumentException e ) {
logWarn( e );
}
- catch ( InvocationTargetException e ) {
+ catch ( final InvocationTargetException e ) {
logWarn( e );
}
- catch ( NullPointerException e ) {
+ catch ( final NullPointerException e ) {
logWarn( e );
}
}
@@ -444,30 +444,30 @@ public class BeanMap extends AbstractMap
* @throws ClassCastException if an error occurs creating the method args
*/
@Override
- public Object put(Object name, Object value) throws
IllegalArgumentException, ClassCastException {
+ public Object put(final Object name, final Object value) throws
IllegalArgumentException, ClassCastException {
if ( bean != null ) {
- Object oldValue = get( name );
- Method method = getWriteMethod( name );
+ final Object oldValue = get( name );
+ final Method method = getWriteMethod( name );
if ( method == null ) {
throw new IllegalArgumentException( "The bean of type: "+
bean.getClass().getName() + " has no property called:
" + name );
}
try {
- Object[] arguments = createWriteMethodArguments( method, value
);
+ final Object[] arguments = createWriteMethodArguments( method,
value );
method.invoke( bean, arguments );
- Object newValue = get( name );
+ final Object newValue = get( name );
firePropertyChange( name, oldValue, newValue );
}
- catch ( InvocationTargetException e ) {
- IllegalArgumentException iae = new
IllegalArgumentException(e.getMessage());
+ catch ( final InvocationTargetException e ) {
+ final IllegalArgumentException iae = new
IllegalArgumentException(e.getMessage());
if (BeanUtils.initCause(iae, e) == false) {
logInfo(e);
}
throw iae;
}
- catch ( IllegalAccessException e ) {
- IllegalArgumentException iae = new
IllegalArgumentException(e.getMessage());
+ catch ( final IllegalAccessException e ) {
+ final IllegalArgumentException iae = new
IllegalArgumentException(e.getMessage());
if (BeanUtils.initCause(iae, e) == false) {
logInfo(e);
}
@@ -536,8 +536,8 @@ public class BeanMap extends AbstractMap
*/
@Override
public Collection<Object> values() {
- ArrayList<Object> answer = new ArrayList<Object>( readMethods.size() );
- for ( Iterator<Object> iter = valueIterator(); iter.hasNext(); ) {
+ final ArrayList<Object> answer = new ArrayList<Object>(
readMethods.size() );
+ for ( final Iterator<Object> iter = valueIterator(); iter.hasNext(); )
{
answer.add( iter.next() );
}
return Collections.unmodifiableList(answer);
@@ -554,7 +554,7 @@ public class BeanMap extends AbstractMap
* @return the type of the property, or <code>null</code> if no such
* property exists
*/
- public Class<?> getType(String name) {
+ public Class<?> getType(final String name) {
return types.get( name );
}
@@ -581,7 +581,7 @@ public class BeanMap extends AbstractMap
return iter.hasNext();
}
public Object next() {
- Object key = iter.next();
+ final Object key = iter.next();
return get(key);
}
public void remove() {
@@ -602,9 +602,10 @@ public class BeanMap extends AbstractMap
return iter.hasNext();
}
public Map.Entry<Object, Object> next() {
- Object key = iter.next();
- Object value = get(key);
+ final Object key = iter.next();
+ final Object value = get(key);
@SuppressWarnings("unchecked")
+ final
// This should not cause any problems; the key is actually a
// string, but it does no harm to expose it as Object
Map.Entry<Object, Object> tmpEntry = new Entry( BeanMap.this,
key, value );
@@ -636,7 +637,7 @@ public class BeanMap extends AbstractMap
*
* @param newBean the new bean to operate on
*/
- public void setBean( Object newBean ) {
+ public void setBean( final Object newBean ) {
bean = newBean;
reinitialise();
}
@@ -647,7 +648,7 @@ public class BeanMap extends AbstractMap
* @param name the name of the property
* @return the accessor method for the property, or null
*/
- public Method getReadMethod(String name) {
+ public Method getReadMethod(final String name) {
return readMethods.get(name);
}
@@ -657,7 +658,7 @@ public class BeanMap extends AbstractMap
* @param name the name of the property
* @return the mutator method for the property, or null
*/
- public Method getWriteMethod(String name) {
+ public Method getWriteMethod(final String name) {
return writeMethods.get(name);
}
@@ -673,7 +674,7 @@ public class BeanMap extends AbstractMap
* {@link String}; null if no such property exists; or the accessor
* method for that property
*/
- protected Method getReadMethod( Object name ) {
+ protected Method getReadMethod( final Object name ) {
return readMethods.get( name );
}
@@ -685,7 +686,7 @@ public class BeanMap extends AbstractMap
* {@link String}; null if no such property exists; null if the
* property is read-only; or the mutator method for that property
*/
- protected Method getWriteMethod( Object name ) {
+ protected Method getWriteMethod( final Object name ) {
return writeMethods.get( name );
}
@@ -705,19 +706,19 @@ public class BeanMap extends AbstractMap
return;
}
- Class<? extends Object> beanClass = getBean().getClass();
+ final Class<? extends Object> beanClass = getBean().getClass();
try {
//BeanInfo beanInfo = Introspector.getBeanInfo( bean, null );
- BeanInfo beanInfo = Introspector.getBeanInfo( beanClass );
- PropertyDescriptor[] propertyDescriptors =
beanInfo.getPropertyDescriptors();
+ final BeanInfo beanInfo = Introspector.getBeanInfo( beanClass );
+ final PropertyDescriptor[] propertyDescriptors =
beanInfo.getPropertyDescriptors();
if ( propertyDescriptors != null ) {
for ( int i = 0; i < propertyDescriptors.length; i++ ) {
- PropertyDescriptor propertyDescriptor =
propertyDescriptors[i];
+ final PropertyDescriptor propertyDescriptor =
propertyDescriptors[i];
if ( propertyDescriptor != null ) {
- String name = propertyDescriptor.getName();
- Method readMethod = propertyDescriptor.getReadMethod();
- Method writeMethod =
propertyDescriptor.getWriteMethod();
- Class<? extends Object> aType =
propertyDescriptor.getPropertyType();
+ final String name = propertyDescriptor.getName();
+ final Method readMethod =
propertyDescriptor.getReadMethod();
+ final Method writeMethod =
propertyDescriptor.getWriteMethod();
+ final Class<? extends Object> aType =
propertyDescriptor.getPropertyType();
if ( readMethod != null ) {
readMethods.put( name, readMethod );
@@ -730,7 +731,7 @@ public class BeanMap extends AbstractMap
}
}
}
- catch ( IntrospectionException e ) {
+ catch ( final IntrospectionException e ) {
logWarn( e );
}
}
@@ -744,7 +745,7 @@ public class BeanMap extends AbstractMap
* @param oldValue the old value for that property
* @param newValue the new value for that property
*/
- protected void firePropertyChange( Object key, Object oldValue, Object
newValue ) {
+ protected void firePropertyChange( final Object key, final Object
oldValue, final Object newValue ) {
}
// Implementation classes
@@ -763,7 +764,7 @@ public class BeanMap extends AbstractMap
* @param key the key for this entry
* @param value the value for this entry
*/
- protected Entry( BeanMap owner, Object key, Object value ) {
+ protected Entry( final BeanMap owner, final Object key, final Object
value ) {
super( key, value );
this.owner = owner;
}
@@ -775,12 +776,12 @@ public class BeanMap extends AbstractMap
* @return the old value for the entry
*/
@Override
- public Object setValue(Object value) {
- Object key = getKey();
- Object oldValue = owner.get( key );
+ public Object setValue(final Object value) {
+ final Object key = getKey();
+ final Object oldValue = owner.get( key );
owner.put( key, value );
- Object newValue = owner.get( key );
+ final Object newValue = owner.get( key );
super.setValue( newValue );
return oldValue;
}
@@ -801,30 +802,30 @@ public class BeanMap extends AbstractMap
* by {@link #convertType(Class,Object)}
* @throws ClassCastException if an error occurs creating the method args
*/
- protected Object[] createWriteMethodArguments( Method method, Object value
)
+ protected Object[] createWriteMethodArguments( final Method method, Object
value )
throws IllegalAccessException, ClassCastException {
try {
if ( value != null ) {
- Class<? extends Object>[] types = method.getParameterTypes();
+ final Class<? extends Object>[] types =
method.getParameterTypes();
if ( types != null && types.length > 0 ) {
- Class<? extends Object> paramType = types[0];
+ final Class<? extends Object> paramType = types[0];
if ( ! paramType.isAssignableFrom( value.getClass() ) ) {
value = convertType( paramType, value );
}
}
}
- Object[] answer = { value };
+ final Object[] answer = { value };
return answer;
}
- catch ( InvocationTargetException e ) {
- IllegalArgumentException iae = new
IllegalArgumentException(e.getMessage());
+ catch ( final InvocationTargetException e ) {
+ final IllegalArgumentException iae = new
IllegalArgumentException(e.getMessage());
if (BeanUtils.initCause(iae, e) == false) {
logInfo(e);
}
throw iae;
}
- catch ( InstantiationException e ) {
- IllegalArgumentException iae = new
IllegalArgumentException(e.getMessage());
+ catch ( final InstantiationException e ) {
+ final IllegalArgumentException iae = new
IllegalArgumentException(e.getMessage());
if (BeanUtils.initCause(iae, e) == false) {
logInfo(e);
}
@@ -864,19 +865,19 @@ public class BeanMap extends AbstractMap
* @throws IllegalAccessException never
* @throws IllegalArgumentException never
*/
- protected Object convertType( Class<?> newType, Object value )
+ protected Object convertType( final Class<?> newType, final Object value )
throws InstantiationException, IllegalAccessException,
IllegalArgumentException, InvocationTargetException {
// try call constructor
- Class<?>[] types = { value.getClass() };
+ final Class<?>[] types = { value.getClass() };
try {
- Constructor<?> constructor = newType.getConstructor( types );
- Object[] arguments = { value };
+ final Constructor<?> constructor = newType.getConstructor( types );
+ final Object[] arguments = { value };
return constructor.newInstance( arguments );
}
- catch ( NoSuchMethodException e ) {
+ catch ( final NoSuchMethodException e ) {
// try using the transformers
- Transformer transformer = getTypeTransformer( newType );
+ final Transformer transformer = getTypeTransformer( newType );
if ( transformer != null ) {
return transformer.transform( value );
}
@@ -891,7 +892,7 @@ public class BeanMap extends AbstractMap
* @return a transformer that will convert strings into that type,
* or null if the given type is not a primitive type
*/
- protected Transformer getTypeTransformer( Class<?> aType ) {
+ protected Transformer getTypeTransformer( final Class<?> aType ) {
return typeTransformers.get( aType );
}
@@ -901,7 +902,7 @@ public class BeanMap extends AbstractMap
*
* @param ex the exception to log
*/
- protected void logInfo(Exception ex) {
+ protected void logInfo(final Exception ex) {
// Deliberately do not use LOG4J or Commons Logging to avoid
dependencies
System.out.println( "INFO: Exception: " + ex );
}
@@ -912,7 +913,7 @@ public class BeanMap extends AbstractMap
*
* @param ex the exception to log
*/
- protected void logWarn(Exception ex) {
+ protected void logWarn(final Exception ex) {
// Deliberately do not use LOG4J or Commons Logging to avoid
dependencies
System.out.println( "WARN: Exception: " + ex );
ex.printStackTrace();
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPredicate.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPredicate.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPredicate.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPredicate.java
Wed Oct 15 20:15:17 2014
@@ -47,7 +47,7 @@ public class BeanPredicate implements Pr
* @param predicate the <code>Predicate</code> to be applied,
* not null
*/
- public BeanPredicate(String propertyName, Predicate predicate) {
+ public BeanPredicate(final String propertyName, final Predicate predicate)
{
this.propertyName = propertyName;
this.predicate = predicate;
}
@@ -60,26 +60,26 @@ public class BeanPredicate implements Pr
* @return the result of the predicate evaluation
* @throws IllegalArgumentException when the property cannot be evaluated
*/
- public boolean evaluate(Object object) {
+ public boolean evaluate(final Object object) {
boolean evaluation = false;
try {
- Object propValue = PropertyUtils.getProperty( object, propertyName
);
+ final Object propValue = PropertyUtils.getProperty( object,
propertyName );
evaluation = predicate.evaluate(propValue);
- } catch (IllegalArgumentException e) {
+ } catch (final IllegalArgumentException e) {
final String errorMsg = "Problem during evaluation.";
log.error("ERROR: " + errorMsg, e);
throw e;
- } catch (IllegalAccessException e) {
+ } catch (final IllegalAccessException e) {
final String errorMsg = "Unable to access the property provided.";
log.error(errorMsg, e);
throw new IllegalArgumentException(errorMsg);
- } catch (InvocationTargetException e) {
+ } catch (final InvocationTargetException e) {
final String errorMsg = "Exception occurred in property's getter";
log.error(errorMsg, e);
throw new IllegalArgumentException(errorMsg);
- } catch (NoSuchMethodException e) {
+ } catch (final NoSuchMethodException e) {
final String errorMsg = "Property not found.";
log.error(errorMsg, e);
throw new IllegalArgumentException(errorMsg);
@@ -102,7 +102,7 @@ public class BeanPredicate implements Pr
* @param propertyName the name of the property whose value is to be
predicated,
* not null
*/
- public void setPropertyName(String propertyName) {
+ public void setPropertyName(final String propertyName) {
this.propertyName = propertyName;
}
@@ -120,7 +120,7 @@ public class BeanPredicate implements Pr
* during {@link #evaluate(Object)}.
* @param predicate <code>Predicate</code>, not null
*/
- public void setPredicate(Predicate predicate) {
+ public void setPredicate(final Predicate predicate) {
this.predicate = predicate;
}
Modified:
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPropertyValueChangeClosure.java
URL:
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPropertyValueChangeClosure.java?rev=1632171&r1=1632170&r2=1632171&view=diff
==============================================================================
---
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPropertyValueChangeClosure.java
(original)
+++
commons/proper/beanutils/trunk/src/main/java/org/apache/commons/beanutils/BeanPropertyValueChangeClosure.java
Wed Oct 15 20:15:17 2014
@@ -117,7 +117,7 @@ public class BeanPropertyValueChangeClos
* object.
* @throws IllegalArgumentException If the propertyName provided is null
or empty.
*/
- public BeanPropertyValueChangeClosure(String propertyName, Object
propertyValue) {
+ public BeanPropertyValueChangeClosure(final String propertyName, final
Object propertyValue) {
this(propertyName, propertyValue, false);
}
@@ -134,7 +134,7 @@ public class BeanPropertyValueChangeClos
* genenerate an <code>IllegalArgumentException</code> or not.
* @throws IllegalArgumentException If the propertyName provided is null
or empty.
*/
- public BeanPropertyValueChangeClosure(String propertyName, Object
propertyValue, boolean ignoreNull) {
+ public BeanPropertyValueChangeClosure(final String propertyName, final
Object propertyValue, final boolean ignoreNull) {
super();
if ((propertyName != null) && (propertyName.length() > 0)) {
@@ -160,39 +160,39 @@ public class BeanPropertyValueChangeClos
* provided. Or if an object in the property path provided is
<code>null</code> and
* <code>ignoreNull</code> is set to <code>false</code>.
*/
- public void execute(Object object) {
+ public void execute(final Object object) {
try {
PropertyUtils.setProperty(object, propertyName, propertyValue);
- } catch (IllegalArgumentException e) {
+ } catch (final IllegalArgumentException e) {
final String errorMsg = "Unable to execute Closure. Null value
encountered in property path...";
if (ignoreNull) {
log.warn("WARNING: " + errorMsg + e);
} else {
- IllegalArgumentException iae = new
IllegalArgumentException(errorMsg);
+ final IllegalArgumentException iae = new
IllegalArgumentException(errorMsg);
if (!BeanUtils.initCause(iae, e)) {
log.error(errorMsg, e);
}
throw iae;
}
- } catch (IllegalAccessException e) {
+ } catch (final IllegalAccessException e) {
final String errorMsg = "Unable to access the property provided.";
- IllegalArgumentException iae = new
IllegalArgumentException(errorMsg);
+ final IllegalArgumentException iae = new
IllegalArgumentException(errorMsg);
if (!BeanUtils.initCause(iae, e)) {
log.error(errorMsg, e);
}
throw iae;
- } catch (InvocationTargetException e) {
+ } catch (final InvocationTargetException e) {
final String errorMsg = "Exception occurred in property's getter";
- IllegalArgumentException iae = new
IllegalArgumentException(errorMsg);
+ final IllegalArgumentException iae = new
IllegalArgumentException(errorMsg);
if (!BeanUtils.initCause(iae, e)) {
log.error(errorMsg, e);
}
throw iae;
- } catch (NoSuchMethodException e) {
+ } catch (final NoSuchMethodException e) {
final String errorMsg = "Property not found";
- IllegalArgumentException iae = new
IllegalArgumentException(errorMsg);
+ final IllegalArgumentException iae = new
IllegalArgumentException(errorMsg);
if (!BeanUtils.initCause(iae, e)) {
log.error(errorMsg, e);
}
