You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2014/10/15 22:15:22 UTC

svn commit: r1632171 [1/20] - in /commons/proper/beanutils/trunk/src: main/java/org/apache/commons/beanutils/ main/java/org/apache/commons/beanutils/converters/ main/java/org/apache/commons/beanutils/expression/ main/java/org/apache/commons/beanutils/l...

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);
             }



Re: svn commit: r1632171 [1/20] - in /commons/proper/beanutils/trunk/src: main/java/org/apache/commons/beanutils/ main/java/org/apache/commons/beanutils/converters/ main/java/org/apache/commons/beanutils/expression/ main/java/org/apache/commons/beanutils/l...

Posted by Gary Gregory <ga...@gmail.com>.
This is my reasoning:
http://garygregory.wordpress.com/2013/01/26/the-final-kiss-in-java/

Gary

On Thu, Oct 23, 2014 at 3:34 PM, Oliver Heger <ol...@oliver-heger.de>
wrote:

> There was no reaction on my comment so far.
>
> I am tempted to -1 this commit. There is already a report about a patch
> which cannot be applied (comment to [1]).
>
> Oliver
>
> [1] https://issues.apache.org/jira/browse/BEANUTILS-417
>
> Am 16.10.2014 um 21:22 schrieb Oliver Heger:
> > To be honest, I really don't like this commit. My personal sense for
> > aesthetics put aside, you modified almost the whole code base. Patches
> > assigned to Jira tickets will probably no longer apply cleanly.
> >
> > Oliver
> >
> > Am 15.10.2014 um 22:15 schrieb ggregory@apache.org:
> >> 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
> >
> > <snip/>
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>


-- 
E-Mail: garydgregory@gmail.com | ggregory@apache.org
Java Persistence with Hibernate, Second Edition
<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Re: svn commit: r1632171 [1/20] - in /commons/proper/beanutils/trunk/src: main/java/org/apache/commons/beanutils/ main/java/org/apache/commons/beanutils/converters/ main/java/org/apache/commons/beanutils/expression/ main/java/org/apache/commons/beanutils/l...

Posted by Paul Benedict <pb...@apache.org>.
I do not not like "final" used this way. Unless you have a constant or a
language mandating "final" for a particular usage, this is unnecessary
noise to the code.


Cheers,
Paul

On Thu, Oct 23, 2014 at 2:34 PM, Oliver Heger <ol...@oliver-heger.de>
wrote:

> There was no reaction on my comment so far.
>
> I am tempted to -1 this commit. There is already a report about a patch
> which cannot be applied (comment to [1]).
>
> Oliver
>
> [1] https://issues.apache.org/jira/browse/BEANUTILS-417
>
> Am 16.10.2014 um 21:22 schrieb Oliver Heger:
> > To be honest, I really don't like this commit. My personal sense for
> > aesthetics put aside, you modified almost the whole code base. Patches
> > assigned to Jira tickets will probably no longer apply cleanly.
> >
> > Oliver
> >
> > Am 15.10.2014 um 22:15 schrieb ggregory@apache.org:
> >> 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
> >
> > <snip/>
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

Re: svn commit: r1632171 [1/20] - in /commons/proper/beanutils/trunk/src: main/java/org/apache/commons/beanutils/ main/java/org/apache/commons/beanutils/converters/ main/java/org/apache/commons/beanutils/expression/ main/java/org/apache/commons/beanutils/l...

Posted by Oliver Heger <ol...@oliver-heger.de>.
There was no reaction on my comment so far.

I am tempted to -1 this commit. There is already a report about a patch
which cannot be applied (comment to [1]).

Oliver

[1] https://issues.apache.org/jira/browse/BEANUTILS-417

Am 16.10.2014 um 21:22 schrieb Oliver Heger:
> To be honest, I really don't like this commit. My personal sense for
> aesthetics put aside, you modified almost the whole code base. Patches
> assigned to Jira tickets will probably no longer apply cleanly.
> 
> Oliver
> 
> Am 15.10.2014 um 22:15 schrieb ggregory@apache.org:
>> 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
> 
> <snip/>
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: svn commit: r1632171 [1/20] - in /commons/proper/beanutils/trunk/src: main/java/org/apache/commons/beanutils/ main/java/org/apache/commons/beanutils/converters/ main/java/org/apache/commons/beanutils/expression/ main/java/org/apache/commons/beanutils/l...

Posted by Oliver Heger <ol...@oliver-heger.de>.
To be honest, I really don't like this commit. My personal sense for
aesthetics put aside, you modified almost the whole code base. Patches
assigned to Jira tickets will probably no longer apply cleanly.

Oliver

Am 15.10.2014 um 22:15 schrieb ggregory@apache.org:
> 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

<snip/>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org