Remove dead comments

Author: garydgregory

src/main/java/org/apache/commons/beanutils/BeanMap.java

@@ -199,7 +199,6 @@ private static Map<Class<? extends Object>, Transformer> createTypeTransformers(
     private transient HashMap<String, Method> writeMethods = new HashMap<>();
 
     // Constructors
-    //-------------------------------------------------------------------------
 
     private transient HashMap<String, Class<? extends Object>> types = new HashMap<>();
 
@@ -210,7 +209,6 @@ public BeanMap() {
     }
 
     // Map interface
-    //-------------------------------------------------------------------------
 
     /**
      * Constructs a new <code>BeanMap</code> that operates on the
@@ -539,7 +537,6 @@ public Object getBean() {
     }
 
     // Helper methods
-    //-------------------------------------------------------------------------
 
     /**
      * Returns the accessor for the property with the given name.
@@ -586,7 +583,6 @@ protected Transformer getTypeTransformer( final Class<?> aType ) {
     }
 
     // Properties
-    //-------------------------------------------------------------------------
 
     /**
      * Returns the mutator for the property with the given name.
@@ -656,7 +652,6 @@ public Iterator<String> keyIterator() {
     }
 
     // Implementation methods
-    //-------------------------------------------------------------------------
 
     /**
      * Get the keys for this BeanMap.
@@ -750,7 +745,6 @@ public void putAllWriteable(final BeanMap map) {
     }
 
     // Implementation classes
-    //-------------------------------------------------------------------------
 
     /**
      * Reinitializes this bean.  Called during {@link #setBean(Object)}.

src/main/java/org/apache/commons/beanutils/WeakFastHashMap.java

@@ -316,7 +316,6 @@ protected K iteratorNext(final Map.Entry<K, V> entry) {
     }
 
     // Constructors
-    // ----------------------------------------------------------------------
 
     /**
      * Collection implementation over the values of the FastHashMap
@@ -347,7 +346,6 @@ protected V iteratorNext(final Map.Entry<K, V> entry) {
     private boolean fast;
 
     // Property access
-    // ----------------------------------------------------------------------
 
     /**
      * Construct an empty map.
@@ -366,7 +364,6 @@ public WeakFastHashMap(final int capacity) {
     }
 
     // Map access
-    // ----------------------------------------------------------------------
     // These methods can forward straight to the wrapped Map in 'fast' mode.
     // (because they are query methods)
 
@@ -430,7 +427,6 @@ protected Map<K, V> cloneMap(final Map<? extends K, ? extends V> map) {
     }
 
     // Map modification
-    // ----------------------------------------------------------------------
     // These methods perform special behaviour in 'fast' mode.
     // The map is cloned, updated and then assigned back.
     // See the comments at the top as to why this won't always work.
@@ -478,7 +474,6 @@ protected Map<K, V> createMap(final int capacity) {
     }
 
     // Basic object methods
-    // ----------------------------------------------------------------------
 
     protected Map<K, V> createMap(final int capacity, final float factor) {
         return new WeakHashMap<>(capacity, factor);
@@ -499,7 +494,6 @@ public Set<Map.Entry<K, V>> entrySet() {
     }
 
     // Map views
-    // ----------------------------------------------------------------------
 
     /**
      * Compare the specified object with this list for equality.  This
@@ -588,7 +582,6 @@ public boolean getFast() {
     }
 
     // Abstractions on Map creations (for subclasses such as WeakFastHashMap)
-    // ----------------------------------------------------------------------
 
     /**
      * Return the hash code value for this map.  This implementation uses
@@ -685,7 +678,6 @@ public void putAll(final Map<? extends K, ? extends V> in) {
     }
 
     // Map view inner classes
-    // ----------------------------------------------------------------------
 
     /**
      * Remove any mapping for this key, and return any previously

src/test/java/org/apache/commons/beanutils/bugs/Jira347TestCase.java

@@ -120,7 +120,6 @@ public void testMappedPropertyDescriptor_AnyArgsProperty() throws Exception {
         final ClassLoader loader = newClassLoader();
         final Class<?> beanClass    = loader.loadClass(className);
         beanClass.getConstructor().newInstance();
-        // -----------------------------------------------------------------------------
 
         // Sanity checks only
         assertNotNull("ClassLoader is null", loader);

src/test/java/org/apache/commons/beanutils/converters/ArrayConverterTestCase.java

@@ -38,8 +38,6 @@ public static TestSuite suite() {
         return new TestSuite(ArrayConverterTestCase.class);
     }
 
-    // ------------------------------------------------------------------------
-
     /**
      * Construct a new Array Converter test case.
      * @param name Test Name
@@ -74,8 +72,6 @@ private void checkArray(final String msg, final Object expected, final Object re
     public void setUp() throws Exception {
     }
 
-    // ------------------------------------------------------------------------
-
     /** Tear Down */
     @Override
     public void tearDown() throws Exception {

src/test/java/org/apache/commons/beanutils/converters/BigDecimalConverterTestCase.java

@@ -42,14 +42,10 @@ private ExtendingBigDecimal(final String val) {
         }
     }
 
-    // ------------------------------------------------------------------------
-
     public static TestSuite suite() {
         return new TestSuite(BigDecimalConverterTestCase.class);
     }
 
-    // ------------------------------------------------------------------------
-
     private Converter converter;
 
     public BigDecimalConverterTestCase(final String name) {
@@ -61,8 +57,6 @@ protected Class<?> getExpectedType() {
         return BigDecimal.class;
     }
 
-    // ------------------------------------------------------------------------
-
     @Override
     protected NumberConverter makeConverter() {
         return new BigDecimalConverter();
@@ -82,8 +76,6 @@ public void setUp() throws Exception {
         numbers[3] = new BigDecimal("23");
     }
 
-    // ------------------------------------------------------------------------
-
     @Override
     public void tearDown() throws Exception {
         converter = null;

src/test/java/org/apache/commons/beanutils/converters/BigIntegerConverterTestCase.java

@@ -33,12 +33,8 @@ public static TestSuite suite() {
         return new TestSuite(BigIntegerConverterTestCase.class);
     }
 
-    // ------------------------------------------------------------------------
-
     private Converter converter;
 
-    // ------------------------------------------------------------------------
-
     public BigIntegerConverterTestCase(final String name) {
         super(name);
     }
@@ -53,8 +49,6 @@ protected NumberConverter makeConverter() {
         return new BigIntegerConverter();
     }
 
-    // ------------------------------------------------------------------------
-
     @Override
     protected NumberConverter makeConverter(final Object defaultValue) {
         return new BigIntegerConverter(defaultValue);
@@ -74,8 +68,6 @@ public void tearDown() throws Exception {
         converter = null;
     }
 
-    // ------------------------------------------------------------------------
-
     public void testSimpleConversion() throws Exception {
         final String[] message= {
             "from String",

src/test/java/org/apache/commons/beanutils/converters/ByteConverterTestCase.java

@@ -32,12 +32,8 @@ public static TestSuite suite() {
         return new TestSuite(ByteConverterTestCase.class);
     }
 
-    // ------------------------------------------------------------------------
-
     private Converter converter;
 
-    // ------------------------------------------------------------------------
-
     public ByteConverterTestCase(final String name) {
         super(name);
     }
@@ -52,8 +48,6 @@ protected NumberConverter makeConverter() {
         return new ByteConverter();
     }
 
-    // ------------------------------------------------------------------------
-
     @Override
     protected NumberConverter makeConverter(final Object defaultValue) {
         return new ByteConverter(defaultValue);
@@ -72,8 +66,6 @@ public void tearDown() throws Exception {
         converter = null;
     }
 
-    // ------------------------------------------------------------------------
-
     /**
      * Test Invalid Amounts (too big/small)
      */

src/test/java/org/apache/commons/beanutils/converters/CharacterConverterTestCase.java

@@ -36,8 +36,6 @@ public static TestSuite suite() {
         return new TestSuite(CharacterConverterTestCase.class);
     }
 
-    // ------------------------------------------------------------------------
-
     /**
      * Construct a new Character Converter test case.
      * @param name Test Name
@@ -56,8 +54,6 @@ public void setUp() throws Exception {
     public void tearDown() throws Exception {
     }
 
-    // ------------------------------------------------------------------------
-
     /**
      * Tests whether the primitive char class can be passed as target type.
      */

src/test/java/org/apache/commons/beanutils/converters/ClassConverterTestCase.java

@@ -36,8 +36,6 @@ public static TestSuite suite() {
         return new TestSuite(ClassConverterTestCase.class);
     }
 
-    // ------------------------------------------------------------------------
-
     /**
      * Construct a new Class Converter test case.
      * @param name Test Name
@@ -56,8 +54,6 @@ public void setUp() throws Exception {
     public void tearDown() throws Exception {
     }
 
-    // ------------------------------------------------------------------------
-
     /**
      * Test Array Conversion
      */

src/test/java/org/apache/commons/beanutils/converters/ClassReloaderTestCase.java

@@ -27,17 +27,13 @@
 
 public class ClassReloaderTestCase extends TestCase {
 
-    // ------------------------------------------------------------------------
-
     public static class DummyClass {
     }
 
     public static TestSuite suite() {
         return new TestSuite(ClassReloaderTestCase.class);
     }
 
-    // ------------------------------------------------------------------------
-
     public ClassReloaderTestCase(final String name) {
         super(name);
     }