commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ohe...@apache.org
Subject svn commit: r1581069 [2/4] - in /commons/proper/configuration/branches/immutableNodes: ./ src/main/java/org/apache/commons/configuration/ src/main/java/org/apache/commons/configuration/beanutils/ src/main/java/org/apache/commons/configuration/builder/ ...
Date Mon, 24 Mar 2014 21:20:13 GMT
Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/BuilderConfigurationWrapperFactory.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/BuilderConfigurationWrapperFactory.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/BuilderConfigurationWrapperFactory.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/BuilderConfigurationWrapperFactory.java Mon Mar 24 21:20:10 2014
@@ -269,6 +269,7 @@ public class BuilderConfigurationWrapper
          * @return the return value of the method
          * @throws Throwable if an error occurs
          */
+        @Override
         public Object invoke(Object proxy, Method method, Object[] args)
                 throws Throwable
         {

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/CopyObjectDefaultHandler.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/CopyObjectDefaultHandler.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/CopyObjectDefaultHandler.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/CopyObjectDefaultHandler.java Mon Mar 24 21:20:10 2014
@@ -92,6 +92,7 @@ public class CopyObjectDefaultHandler im
      * @throws ConfigurationRuntimeException if an exception occurs
      * @see BuilderParameters#getParameters()
      */
+    @Override
     public void initializeDefaults(Object parameters)
     {
         try

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/DatabaseBuilderParametersImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/DatabaseBuilderParametersImpl.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/DatabaseBuilderParametersImpl.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/DatabaseBuilderParametersImpl.java Mon Mar 24 21:20:10 2014
@@ -60,42 +60,49 @@ public class DatabaseBuilderParametersIm
     /** Constant for the auto commit property. */
     private static final String PROP_AUTO_COMMIT = "autoCommit";
 
+    @Override
     public DatabaseBuilderParametersImpl setDataSource(DataSource src)
     {
         storeProperty(PROP_DATA_SOURCE, src);
         return this;
     }
 
+    @Override
     public DatabaseBuilderParametersImpl setTable(String tname)
     {
         storeProperty(PROP_TABLE, tname);
         return this;
     }
 
+    @Override
     public DatabaseBuilderParametersImpl setKeyColumn(String name)
     {
         storeProperty(PROP_KEY_COLUMN, name);
         return this;
     }
 
+    @Override
     public DatabaseBuilderParametersImpl setValueColumn(String name)
     {
         storeProperty(PROP_VALUE_COLUMN, name);
         return this;
     }
 
+    @Override
     public DatabaseBuilderParametersImpl setConfigurationNameColumn(String name)
     {
         storeProperty(PROP_CONFIG_NAME_COLUMN, name);
         return this;
     }
 
+    @Override
     public DatabaseBuilderParametersImpl setConfigurationName(String name)
     {
         storeProperty(PROP_CONFIG_NAME, name);
         return this;
     }
 
+    @Override
     public DatabaseBuilderParametersImpl setAutoCommit(boolean f)
     {
         storeProperty(PROP_AUTO_COMMIT, Boolean.valueOf(f));

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/DefaultReloadingDetectorFactory.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/DefaultReloadingDetectorFactory.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/DefaultReloadingDetectorFactory.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/DefaultReloadingDetectorFactory.java Mon Mar 24 21:20:10 2014
@@ -36,6 +36,7 @@ import org.apache.commons.configuration.
 public class DefaultReloadingDetectorFactory implements
         ReloadingDetectorFactory
 {
+    @Override
     public ReloadingDetector createReloadingDetector(FileHandler handler,
             FileBasedBuilderParametersImpl params)
             throws ConfigurationException

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/FileBasedBuilderParametersImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/FileBasedBuilderParametersImpl.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/FileBasedBuilderParametersImpl.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/FileBasedBuilderParametersImpl.java Mon Mar 24 21:20:10 2014
@@ -146,6 +146,7 @@ public class FileBasedBuilderParametersI
         return reloadingRefreshDelay;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setReloadingRefreshDelay(
             Long reloadingRefreshDelay)
     {
@@ -164,6 +165,7 @@ public class FileBasedBuilderParametersI
         return reloadingDetectorFactory;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setReloadingDetectorFactory(
             ReloadingDetectorFactory reloadingDetectorFactory)
     {
@@ -171,42 +173,49 @@ public class FileBasedBuilderParametersI
         return this;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setFile(File file)
     {
         getFileHandler().setFile(file);
         return this;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setURL(URL url)
     {
         getFileHandler().setURL(url);
         return this;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setPath(String path)
     {
         getFileHandler().setPath(path);
         return this;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setFileName(String name)
     {
         getFileHandler().setFileName(name);
         return this;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setBasePath(String path)
     {
         getFileHandler().setBasePath(path);
         return this;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setFileSystem(FileSystem fs)
     {
         getFileHandler().setFileSystem(fs);
         return this;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setLocationStrategy(
             FileLocationStrategy strategy)
     {
@@ -214,6 +223,7 @@ public class FileBasedBuilderParametersI
         return this;
     }
 
+    @Override
     public FileBasedBuilderParametersImpl setEncoding(String enc)
     {
         getFileHandler().setEncoding(enc);

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/HierarchicalBuilderParametersImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/HierarchicalBuilderParametersImpl.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/HierarchicalBuilderParametersImpl.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/HierarchicalBuilderParametersImpl.java Mon Mar 24 21:20:10 2014
@@ -42,6 +42,7 @@ public class HierarchicalBuilderParamete
      * {@inheritDoc} This implementation stores the expression engine in the
      * internal parameters map.
      */
+    @Override
     public HierarchicalBuilderParametersImpl setExpressionEngine(
             ExpressionEngine engine)
     {

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/JndiBuilderParametersImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/JndiBuilderParametersImpl.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/JndiBuilderParametersImpl.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/JndiBuilderParametersImpl.java Mon Mar 24 21:20:10 2014
@@ -48,12 +48,14 @@ public class JndiBuilderParametersImpl e
     /** Constant for the name of the prefix property. */
     private static final String PROP_PREFIX = "prefix";
 
+    @Override
     public JndiBuilderParametersImpl setContext(Context ctx)
     {
         storeProperty(PROP_CONTEXT, ctx);
         return this;
     }
 
+    @Override
     public JndiBuilderParametersImpl setPrefix(String p)
     {
         storeProperty(PROP_PREFIX, p);

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/PropertiesBuilderParametersImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/PropertiesBuilderParametersImpl.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/PropertiesBuilderParametersImpl.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/PropertiesBuilderParametersImpl.java Mon Mar 24 21:20:10 2014
@@ -52,12 +52,14 @@ public class PropertiesBuilderParameters
     /** The key for the IO factory property. */
     private static final String PROP_IO_FACTORY = "iOFactory";
 
+    @Override
     public PropertiesBuilderParametersImpl setIncludesAllowed(boolean f)
     {
         storeProperty(PROP_INCLUDES_ALLOWED, Boolean.valueOf(f));
         return this;
     }
 
+    @Override
     public PropertiesBuilderParametersImpl setLayout(
             PropertiesConfigurationLayout layout)
     {
@@ -65,6 +67,7 @@ public class PropertiesBuilderParameters
         return this;
     }
 
+    @Override
     public PropertiesBuilderParametersImpl setIOFactory(IOFactory factory)
     {
         storeProperty(PROP_IO_FACTORY, factory);

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/ReloadingFileBasedConfigurationBuilder.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/ReloadingFileBasedConfigurationBuilder.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/ReloadingFileBasedConfigurationBuilder.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/ReloadingFileBasedConfigurationBuilder.java Mon Mar 24 21:20:10 2014
@@ -135,6 +135,7 @@ public class ReloadingFileBasedConfigura
      *
      * @return the {@code ReloadingController}
      */
+    @Override
     public ReloadingController getReloadingController()
     {
         return reloadingController;
@@ -214,6 +215,7 @@ public class ReloadingFileBasedConfigura
     {
         return new ReloadingListener()
         {
+            @Override
             public void reloadingRequired(ReloadingEvent event)
             {
                 resetResult();
@@ -232,6 +234,7 @@ public class ReloadingFileBasedConfigura
     {
         return new ReloadingDetector()
         {
+            @Override
             public void reloadingPerformed()
             {
                 ReloadingDetector detector = resultReloadingDetector;
@@ -241,6 +244,7 @@ public class ReloadingFileBasedConfigura
                 }
             }
 
+            @Override
             public boolean isReloadingRequired()
             {
                 ReloadingDetector detector = resultReloadingDetector;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/XMLBuilderParametersImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/XMLBuilderParametersImpl.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/XMLBuilderParametersImpl.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/XMLBuilderParametersImpl.java Mon Mar 24 21:20:10 2014
@@ -59,6 +59,7 @@ public class XMLBuilderParametersImpl ex
     /** The key for the schema validation flag. */
     private static final String PROP_SCHEMA_VALIDATION = "schemaValidation";
 
+    @Override
     public XMLBuilderParametersImpl setDocumentBuilder(
             DocumentBuilder docBuilder)
     {
@@ -66,6 +67,7 @@ public class XMLBuilderParametersImpl ex
         return this;
     }
 
+    @Override
     public XMLBuilderParametersImpl setEntityResolver(EntityResolver resolver)
     {
         storeProperty(PROP_ENTITY_RESOLVER, resolver);
@@ -83,24 +85,28 @@ public class XMLBuilderParametersImpl ex
         return (EntityResolver) fetchProperty(PROP_ENTITY_RESOLVER);
     }
 
+    @Override
     public XMLBuilderParametersImpl setPublicID(String pubID)
     {
         storeProperty(PROP_PUBLIC_ID, pubID);
         return this;
     }
 
+    @Override
     public XMLBuilderParametersImpl setSystemID(String sysID)
     {
         storeProperty(PROP_SYSTEM_ID, sysID);
         return this;
     }
 
+    @Override
     public XMLBuilderParametersImpl setValidating(boolean f)
     {
         storeProperty(PROP_VALIDATING, Boolean.valueOf(f));
         return this;
     }
 
+    @Override
     public XMLBuilderParametersImpl setSchemaValidation(boolean f)
     {
         storeProperty(PROP_SCHEMA_VALIDATION, Boolean.valueOf(f));

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/BaseConfigurationBuilderProvider.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/BaseConfigurationBuilderProvider.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/BaseConfigurationBuilderProvider.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/BaseConfigurationBuilderProvider.java Mon Mar 24 21:20:10 2014
@@ -167,6 +167,7 @@ public class BaseConfigurationBuilderPro
      * create a new builder instance using reflection and to configure it with
      * parameter values defined by the passed in {@code BeanDeclaration}.
      */
+    @Override
     public ConfigurationBuilder<? extends Configuration> getConfigurationBuilder(
             ConfigurationDeclaration decl) throws ConfigurationException
     {

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/CombinedBuilderParametersImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/CombinedBuilderParametersImpl.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/CombinedBuilderParametersImpl.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/CombinedBuilderParametersImpl.java Mon Mar 24 21:20:10 2014
@@ -142,6 +142,7 @@ public class CombinedBuilderParametersIm
      * @param builder the definition {@code ConfigurationBuilder}
      * @return a reference to this object for method chaining
      */
+    @Override
     public CombinedBuilderParametersImpl setDefinitionBuilder(
             ConfigurationBuilder<? extends HierarchicalConfiguration> builder)
     {
@@ -161,6 +162,7 @@ public class CombinedBuilderParametersIm
      * @return a reference to this object for method chaining
      * @throws IllegalArgumentException if a required parameter is missing
      */
+    @Override
     public CombinedBuilderParametersImpl registerProvider(String tagName,
             ConfigurationBuilderProvider provider)
     {
@@ -282,6 +284,7 @@ public class CombinedBuilderParametersIm
      * @param path the base path for resolving relative file names
      * @return a reference to this object for method chaining
      */
+    @Override
     public CombinedBuilderParametersImpl setBasePath(String path)
     {
         basePath = path;
@@ -312,6 +315,7 @@ public class CombinedBuilderParametersIm
      *        builder
      * @return a reference to this object for method chaining
      */
+    @Override
     public CombinedBuilderParametersImpl setDefinitionBuilderParameters(
             BuilderParameters params)
     {
@@ -357,6 +361,7 @@ public class CombinedBuilderParametersIm
      * demand) is overridden. This also removes all default handlers registered
      * before!
      */
+    @Override
     public CombinedBuilderParametersImpl setChildDefaultParametersManager(
             DefaultParametersManager manager)
     {
@@ -369,6 +374,7 @@ public class CombinedBuilderParametersIm
      * internal {@link DefaultParametersManager} instance. If none was set, a
      * new instance is created now.
      */
+    @Override
     public <D> CombinedBuilderParametersImpl registerChildDefaultsHandler(
             Class<D> paramClass, DefaultParametersHandler<? super D> handler)
     {
@@ -382,6 +388,7 @@ public class CombinedBuilderParametersIm
      * internal {@link DefaultParametersManager} instance. If none was set, a
      * new instance is created now.
      */
+    @Override
     public <D> CombinedBuilderParametersImpl registerChildDefaultsHandler(
             Class<D> paramClass, DefaultParametersHandler<? super D> handler,
             Class<?> startClass)

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/CombinedConfigurationBuilder.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/CombinedConfigurationBuilder.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/CombinedConfigurationBuilder.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/CombinedConfigurationBuilder.java Mon Mar 24 21:20:10 2014
@@ -1287,6 +1287,7 @@ public class CombinedConfigurationBuilde
     {
         defBuilder.addBuilderListener(new BuilderListener()
         {
+            @Override
             public void builderReset(
                     ConfigurationBuilder<? extends Configuration> builder)
             {
@@ -1570,6 +1571,7 @@ public class CombinedConfigurationBuilde
         {
             changeListener = new BuilderListener()
             {
+                @Override
                 public void builderReset(
                         ConfigurationBuilder<? extends Configuration> builder)
                 {
@@ -1598,9 +1600,8 @@ public class CombinedConfigurationBuilde
                     .hasNext();)
             {
                 String nodeName = it.next().getRootElementName();
-                for (int i = 0; i < CONFIG_SECTIONS.length; i++)
-                {
-                    if (CONFIG_SECTIONS[i].equals(nodeName))
+                for (String element : CONFIG_SECTIONS) {
+                    if (element.equals(nodeName))
                     {
                         it.remove();
                         break;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileBuilderParametersImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileBuilderParametersImpl.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileBuilderParametersImpl.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileBuilderParametersImpl.java Mon Mar 24 21:20:10 2014
@@ -119,6 +119,7 @@ public class MultiFileBuilderParametersI
         return filePattern;
     }
 
+    @Override
     public MultiFileBuilderParametersImpl setFilePattern(String p)
     {
         filePattern = p;
@@ -135,6 +136,7 @@ public class MultiFileBuilderParametersI
         return managedBuilderParameters;
     }
 
+    @Override
     public MultiFileBuilderParametersImpl setManagedBuilderParameters(
             BuilderParameters p)
     {

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileConfigurationBuilder.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileConfigurationBuilder.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileConfigurationBuilder.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileConfigurationBuilder.java Mon Mar 24 21:20:10 2014
@@ -109,6 +109,7 @@ public class MultiFileConfigurationBuild
     private final BuilderListener managedBuilderDelegationListener =
             new BuilderListener()
             {
+                @Override
                 public void builderReset(
                         ConfigurationBuilder<? extends Configuration> builder)
                 {

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileConfigurationBuilderProvider.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileConfigurationBuilderProvider.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileConfigurationBuilderProvider.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiFileConfigurationBuilderProvider.java Mon Mar 24 21:20:10 2014
@@ -174,16 +174,19 @@ public class MultiFileConfigurationBuild
             builder = bldr;
         }
 
+        @Override
         public Configuration getConfiguration() throws ConfigurationException
         {
             return configuration;
         }
 
+        @Override
         public void addBuilderListener(BuilderListener l)
         {
             builder.addBuilderListener(l);
         }
 
+        @Override
         public void removeBuilderListener(BuilderListener l)
         {
             builder.removeBuilderListener(l);
@@ -216,6 +219,7 @@ public class MultiFileConfigurationBuild
             ctrlSupport = (ReloadingControllerSupport) bldr;
         }
 
+        @Override
         public ReloadingController getReloadingController()
         {
             return ctrlSupport.getReloadingController();

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiWrapDynaBean.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiWrapDynaBean.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiWrapDynaBean.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiWrapDynaBean.java Mon Mar 24 21:20:10 2014
@@ -87,22 +87,26 @@ class MultiWrapDynaBean implements DynaB
      * objects used internally by this class. Therefore, just an exception is
      * thrown.
      */
+    @Override
     public boolean contains(String name, String key)
     {
         throw new UnsupportedOperationException(
                 "contains() operation not supported!");
     }
 
+    @Override
     public Object get(String name)
     {
         return fetchBean(name).get(name);
     }
 
+    @Override
     public Object get(String name, int index)
     {
         return fetchBean(name).get(name, index);
     }
 
+    @Override
     public Object get(String name, String key)
     {
         return fetchBean(name).get(name, key);
@@ -112,6 +116,7 @@ class MultiWrapDynaBean implements DynaB
      * {@inheritDoc} This implementation returns an instance of
      * {@code MultiWrapDynaClass}.
      */
+    @Override
     public DynaClass getDynaClass()
     {
         return dynaClass;
@@ -122,22 +127,26 @@ class MultiWrapDynaBean implements DynaB
      * objects used internally by this class. Therefore, just an exception is
      * thrown.
      */
+    @Override
     public void remove(String name, String key)
     {
         throw new UnsupportedOperationException(
                 "remove() operation not supported!");
     }
 
+    @Override
     public void set(String name, Object value)
     {
         fetchBean(name).set(name, value);
     }
 
+    @Override
     public void set(String name, int index, Object value)
     {
         fetchBean(name).set(name, index, value);
     }
 
+    @Override
     public void set(String name, String key, Object value)
     {
         fetchBean(name).set(name, key, value);

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiWrapDynaClass.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiWrapDynaClass.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiWrapDynaClass.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/MultiWrapDynaClass.java Mon Mar 24 21:20:10 2014
@@ -61,16 +61,19 @@ class MultiWrapDynaClass implements Dyna
     /**
      * {@inheritDoc} The name of this class is not relevant.
      */
+    @Override
     public String getName()
     {
         return null;
     }
 
+    @Override
     public DynaProperty getDynaProperty(String name)
     {
         return namedProperties.get(name);
     }
 
+    @Override
     public DynaProperty[] getDynaProperties()
     {
         return properties.toArray(EMPTY_PROPS);
@@ -80,6 +83,7 @@ class MultiWrapDynaClass implements Dyna
      * {@inheritDoc} This implementation always throws an exception because it
      * is not possible to instantiate a bean of multiple classes.
      */
+    @Override
     public DynaBean newInstance() throws IllegalAccessException,
             InstantiationException
     {

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/ReloadingCombinedConfigurationBuilder.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/ReloadingCombinedConfigurationBuilder.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/ReloadingCombinedConfigurationBuilder.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/ReloadingCombinedConfigurationBuilder.java Mon Mar 24 21:20:10 2014
@@ -106,6 +106,7 @@ public class ReloadingCombinedConfigurat
      * for the first time). So this method does not return a meaningful result
      * before.
      */
+    @Override
     public synchronized ReloadingController getReloadingController()
     {
         return reloadingController;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/ReloadingMultiFileConfigurationBuilder.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/ReloadingMultiFileConfigurationBuilder.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/ReloadingMultiFileConfigurationBuilder.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/combined/ReloadingMultiFileConfigurationBuilder.java Mon Mar 24 21:20:10 2014
@@ -118,6 +118,7 @@ public class ReloadingMultiFileConfigura
      * {@code ReloadingController} that delegates to the reloading controllers
      * of the managed builders created so far.
      */
+    @Override
     public ReloadingController getReloadingController()
     {
         return reloadingController;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/fluent/Parameters.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/fluent/Parameters.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/fluent/Parameters.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/builder/fluent/Parameters.java Mon Mar 24 21:20:10 2014
@@ -329,6 +329,7 @@ public final class Parameters
          * {@inheritDoc} This implementation delegates method invocations to the
          * target object and handles the return value correctly.
          */
+        @Override
         public Object invoke(Object proxy, Method method, Object[] args)
                 throws Throwable
         {

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DefaultConversionHandler.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DefaultConversionHandler.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DefaultConversionHandler.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DefaultConversionHandler.java Mon Mar 24 21:20:10 2014
@@ -109,6 +109,7 @@ public class DefaultConversionHandler im
         this.dateFormat = dateFormat;
     }
 
+    @Override
     public <T> T to(Object src, Class<T> targetCls, ConfigurationInterpolator ci)
     {
         ConfigurationInterpolator interpolator = fetchInterpolator(ci);
@@ -122,6 +123,7 @@ public class DefaultConversionHandler im
      * supported. If the source object is <b>null</b>, result is <b>null</b>,
      * too.
      */
+    @Override
     public Object toArray(Object src, Class<?> elemClass,
             ConfigurationInterpolator ci)
     {
@@ -147,6 +149,7 @@ public class DefaultConversionHandler im
      *
      * @throws IllegalArgumentException if the target collection is <b>null</b>
      */
+    @Override
     public <T> void toCollection(Object src, Class<T> elemClass,
             ConfigurationInterpolator ci, Collection<T> dest)
     {

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DefaultListDelimiterHandler.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DefaultListDelimiterHandler.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DefaultListDelimiterHandler.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DefaultListDelimiterHandler.java Mon Mar 24 21:20:10 2014
@@ -101,6 +101,7 @@ public class DefaultListDelimiterHandler
         return delimiter;
     }
 
+    @Override
     public Object escapeList(List<?> values, ValueTransformer transformer)
     {
         Object[] escapedValues = new String[values.size()];

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DisabledListDelimiterHandler.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DisabledListDelimiterHandler.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DisabledListDelimiterHandler.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/DisabledListDelimiterHandler.java Mon Mar 24 21:20:10 2014
@@ -55,6 +55,7 @@ public class DisabledListDelimiterHandle
      * {@inheritDoc} This implementation always throws an
      * {@code UnsupportedOperationException} exception.
      */
+    @Override
     public Object escapeList(List<?> values, ValueTransformer transformer)
     {
         throw new UnsupportedOperationException(

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/LegacyListDelimiterHandler.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/LegacyListDelimiterHandler.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/LegacyListDelimiterHandler.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/LegacyListDelimiterHandler.java Mon Mar 24 21:20:10 2014
@@ -102,6 +102,7 @@ public class LegacyListDelimiterHandler 
      * backslashes at the end of a string so that they are not interpreted as
      * escape character for a following list delimiter.
      */
+    @Override
     public Object escapeList(List<?> values, ValueTransformer transformer)
     {
         if (!values.isEmpty())

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/ListDelimiterHandler.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/ListDelimiterHandler.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/ListDelimiterHandler.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/convert/ListDelimiterHandler.java Mon Mar 24 21:20:10 2014
@@ -62,6 +62,7 @@ public interface ListDelimiterHandler
      */
     ValueTransformer NOOP_TRANSFORMER = new ValueTransformer()
     {
+        @Override
         public Object transformValue(Object value)
         {
             return value;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/BaseEventSource.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/BaseEventSource.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/BaseEventSource.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/BaseEventSource.java Mon Mar 24 21:20:10 2014
@@ -89,12 +89,14 @@ public class BaseEventSource implements 
         initListeners();
     }
 
+    @Override
     public void addConfigurationListener(ConfigurationListener l)
     {
         checkListener(l);
         listeners.add(l);
     }
 
+    @Override
     public boolean removeConfigurationListener(ConfigurationListener l)
     {
         return listeners.remove(l);
@@ -156,12 +158,14 @@ public class BaseEventSource implements 
         }
     }
 
+    @Override
     public void addErrorListener(ConfigurationErrorListener l)
     {
         checkListener(l);
         errorListeners.add(l);
     }
 
+    @Override
     public boolean removeErrorListener(ConfigurationErrorListener l)
     {
         return errorListeners.remove(l);

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/ConfigurationErrorEvent.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/ConfigurationErrorEvent.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/ConfigurationErrorEvent.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/ConfigurationErrorEvent.java Mon Mar 24 21:20:10 2014
@@ -62,7 +62,7 @@ public class ConfigurationErrorEvent ext
     private static final long serialVersionUID = -7433184493062648409L;
 
     /** Stores the exception that caused this event. */
-    private Throwable cause;
+    private final Throwable cause;
 
     /**
      * Creates a new instance of {@code ConfigurationErrorEvent} and

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/ConfigurationEvent.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/ConfigurationEvent.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/ConfigurationEvent.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/event/ConfigurationEvent.java Mon Mar 24 21:20:10 2014
@@ -83,16 +83,16 @@ public class ConfigurationEvent extends 
     private static final long serialVersionUID = 3277238219073504136L;
 
     /** Stores the event type. */
-    private int type;
+    private final int type;
 
     /** Stores the property name. */
-    private String propertyName;
+    private final String propertyName;
 
     /** Stores the property value. */
-    private Object propertyValue;
+    private final Object propertyValue;
 
     /** Stores the before update flag. */
-    private boolean beforeUpdate;
+    private final boolean beforeUpdate;
 
     /**
      * Creates a new instance of {@code ConfigurationEvent} and

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/ConstantLookup.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/ConstantLookup.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/ConstantLookup.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/ConstantLookup.java Mon Mar 24 21:20:10 2014
@@ -60,7 +60,7 @@ public class ConstantLookup implements L
     private static Map<String, Object> constantCache = new HashMap<String, Object>();
 
     /** The logger. */
-    private Log log = LogFactory.getLog(getClass());
+    private final Log log = LogFactory.getLog(getClass());
 
     /**
      * Tries to resolve the specified variable. The passed in variable name is
@@ -74,6 +74,7 @@ public class ConstantLookup implements L
      * @return the value of this variable or <b>null</b> if it cannot be
      * resolved
      */
+    @Override
     public Object lookup(String var)
     {
         if (var == null)

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/DummyLookup.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/DummyLookup.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/DummyLookup.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/DummyLookup.java Mon Mar 24 21:20:10 2014
@@ -42,6 +42,7 @@ public enum DummyLookup implements Looku
     /**
      * {@inheritDoc} This implementation always returns <b>null</b>.
      */
+    @Override
     public Object lookup(String variable)
     {
         return null;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/EnvironmentLookup.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/EnvironmentLookup.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/EnvironmentLookup.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/EnvironmentLookup.java Mon Mar 24 21:20:10 2014
@@ -52,6 +52,7 @@ public class EnvironmentLookup implement
      * @param key the key to lookup
      * @return the value of this key or <b>null</b> if it cannot be resolved
      */
+    @Override
     public String lookup(String key)
     {
         return System.getenv(key);

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/ExprLookup.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/ExprLookup.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/ExprLookup.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/ExprLookup.java Mon Mar 24 21:20:10 2014
@@ -216,6 +216,7 @@ public class ExprLookup implements Looku
      * @param var The expression.
      * @return The String result of the expression.
      */
+    @Override
     public String lookup(String var)
     {
         if(substitutor == null)

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/SystemPropertiesLookup.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/SystemPropertiesLookup.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/SystemPropertiesLookup.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/interpol/SystemPropertiesLookup.java Mon Mar 24 21:20:10 2014
@@ -48,6 +48,7 @@ public class SystemPropertiesLookup impl
     private final StrLookup<String> sysLookup = StrLookup
             .systemPropertiesLookup();
 
+    @Override
     public Object lookup(String variable)
     {
         return sysLookup.lookup(variable);

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/AbsoluteNameLocationStrategy.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/AbsoluteNameLocationStrategy.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/AbsoluteNameLocationStrategy.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/AbsoluteNameLocationStrategy.java Mon Mar 24 21:20:10 2014
@@ -42,6 +42,7 @@ public class AbsoluteNameLocationStrateg
      * the locator's file name (if defined). If this results in an absolute file
      * name pointing to an existing file, the corresponding URL is returned.
      */
+    @Override
     public URL locate(FileSystem fileSystem, FileLocator locator)
     {
         if (StringUtils.isNotEmpty(locator.getFileName()))

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/BasePathLocationStrategy.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/BasePathLocationStrategy.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/BasePathLocationStrategy.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/BasePathLocationStrategy.java Mon Mar 24 21:20:10 2014
@@ -44,6 +44,7 @@ public class BasePathLocationStrategy im
      * base path and file name. If this {@code File} exists, its URL is
      * returned.
      */
+    @Override
     public URL locate(FileSystem fileSystem, FileLocator locator)
     {
         if (StringUtils.isNotEmpty(locator.getFileName()))

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/ClasspathLocationStrategy.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/ClasspathLocationStrategy.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/ClasspathLocationStrategy.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/ClasspathLocationStrategy.java Mon Mar 24 21:20:10 2014
@@ -40,6 +40,7 @@ public class ClasspathLocationStrategy i
      * {@inheritDoc} This implementation looks up the locator's file name as a
      * resource on the class path.
      */
+    @Override
     public URL locate(FileSystem fileSystem, FileLocator locator)
     {
         return StringUtils.isEmpty(locator.getFileName()) ? null

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/CombinedLocationStrategy.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/CombinedLocationStrategy.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/CombinedLocationStrategy.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/CombinedLocationStrategy.java Mon Mar 24 21:20:10 2014
@@ -96,6 +96,7 @@ public class CombinedLocationStrategy im
      * {@inheritDoc} This implementation tries to locate the file by delegating
      * to the managed sub strategies.
      */
+    @Override
     public URL locate(FileSystem fileSystem, FileLocator locator)
     {
         for (FileLocationStrategy sub : getSubStrategies())

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileHandler.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileHandler.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileHandler.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileHandler.java Mon Mar 24 21:20:10 2014
@@ -132,18 +132,22 @@ public class FileHandler
     private static final SynchronizerSupport DUMMY_SYNC_SUPPORT =
             new SynchronizerSupport()
             {
+                @Override
                 public void unlock(LockMode mode)
                 {
                 }
 
+                @Override
                 public void setSynchronizer(Synchronizer sync)
                 {
                 }
 
+                @Override
                 public void lock(LockMode mode)
                 {
                 }
 
+                @Override
                 public Synchronizer getSynchronizer()
                 {
                     return NoOpSynchronizer.INSTANCE;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileHandlerListenerAdapter.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileHandlerListenerAdapter.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileHandlerListenerAdapter.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileHandlerListenerAdapter.java Mon Mar 24 21:20:10 2014
@@ -33,22 +33,27 @@ package org.apache.commons.configuration
  */
 public class FileHandlerListenerAdapter implements FileHandlerListener
 {
+    @Override
     public void loading(FileHandler handler)
     {
     }
 
+    @Override
     public void loaded(FileHandler handler)
     {
     }
 
+    @Override
     public void saving(FileHandler handler)
     {
     }
 
+    @Override
     public void saved(FileHandler handler)
     {
     }
 
+    @Override
     public void locationChanged(FileHandler handler)
     {
     }

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileSystemLocationStrategy.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileSystemLocationStrategy.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileSystemLocationStrategy.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/FileSystemLocationStrategy.java Mon Mar 24 21:20:10 2014
@@ -39,6 +39,7 @@ public class FileSystemLocationStrategy 
     /**
      * {@inheritDoc} This implementation delegates to the {@code FileSystem}.
      */
+    @Override
     public URL locate(FileSystem fileSystem, FileLocator locator)
     {
         return fileSystem.locateFromURL(locator.getBasePath(),

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/HomeDirectoryLocationStrategy.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/HomeDirectoryLocationStrategy.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/HomeDirectoryLocationStrategy.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/HomeDirectoryLocationStrategy.java Mon Mar 24 21:20:10 2014
@@ -120,6 +120,7 @@ public class HomeDirectoryLocationStrate
      * defines a base path and the {@code evaluateBasePath} property is
      * <b>true</b>, a sub directory of the home directory is searched.
      */
+    @Override
     public URL locate(FileSystem fileSystem, FileLocator locator)
     {
         if (StringUtils.isNotEmpty(locator.getFileName()))

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/ProvidedURLLocationStrategy.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/ProvidedURLLocationStrategy.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/ProvidedURLLocationStrategy.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/io/ProvidedURLLocationStrategy.java Mon Mar 24 21:20:10 2014
@@ -40,6 +40,7 @@ public class ProvidedURLLocationStrategy
      * {@inheritDoc} This implementation just returns the URL stored in the
      * given {@code FileLocator}.
      */
+    @Override
     public URL locate(FileSystem fileSystem, FileLocator locator)
     {
         return locator.getSourceURL();

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/plist/PropertyListConfiguration.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/plist/PropertyListConfiguration.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/plist/PropertyListConfiguration.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/plist/PropertyListConfiguration.java Mon Mar 24 21:20:10 2014
@@ -187,6 +187,7 @@ public class PropertyListConfiguration e
         }
     }
 
+    @Override
     public void read(Reader in) throws ConfigurationException
     {
         PropertyListParser parser = new PropertyListParser(in);
@@ -201,6 +202,7 @@ public class PropertyListConfiguration e
         }
     }
 
+    @Override
     public void write(Writer out) throws ConfigurationException
     {
         PrintWriter writer = new PrintWriter(out);
@@ -421,9 +423,8 @@ public class PropertyListConfiguration e
     {
         StringBuilder buf = new StringBuilder();
 
-        for (int i = 0; i < DATE_PARSERS.length; i++)
-        {
-            DATE_PARSERS[i].formatComponent(buf, cal);
+        for (DateComponentParser element : DATE_PARSERS) {
+            element.formatComponent(buf, cal);
         }
 
         return buf.toString();
@@ -537,13 +538,13 @@ public class PropertyListConfiguration e
     private static class DateFieldParser extends DateComponentParser
     {
         /** Stores the calendar field to be processed. */
-        private int calendarField;
+        private final int calendarField;
 
         /** Stores the length of this field. */
-        private int length;
+        private final int length;
 
         /** An optional offset to add to the calendar field. */
-        private int offset;
+        private final int offset;
 
         /**
          * Creates a new instance of {@code DateFieldParser}.
@@ -604,7 +605,7 @@ public class PropertyListConfiguration e
     private static class DateSeparatorParser extends DateComponentParser
     {
         /** Stores the separator. */
-        private String separator;
+        private final String separator;
 
         /**
          * Creates a new instance of {@code DateSeparatorParser} and sets

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/plist/XMLPropertyListConfiguration.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/plist/XMLPropertyListConfiguration.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/plist/XMLPropertyListConfiguration.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/plist/XMLPropertyListConfiguration.java Mon Mar 24 21:20:10 2014
@@ -201,11 +201,13 @@ public class XMLPropertyListConfiguratio
      *
      * @param locator the current {@code FileLocator}
      */
+    @Override
     public void initFileLocator(FileLocator locator)
     {
         this.locator = locator;
     }
 
+    @Override
     public void read(Reader in) throws ConfigurationException
     {
         // We have to make sure that the root node is actually a PListNode.
@@ -219,6 +221,7 @@ public class XMLPropertyListConfiguratio
         // set up the DTD validation
         EntityResolver resolver = new EntityResolver()
         {
+            @Override
             public InputSource resolveEntity(String publicId, String systemId)
             {
                 return new InputSource(getClass().getClassLoader().getResourceAsStream("PropertyList-1.0.dtd"));
@@ -243,6 +246,7 @@ public class XMLPropertyListConfiguratio
         }
     }
 
+    @Override
     public void write(Writer out) throws ConfigurationException
     {
         PrintWriter writer = new PrintWriter(out);
@@ -441,10 +445,10 @@ public class XMLPropertyListConfiguratio
     private class XMLPropertyListHandler extends DefaultHandler
     {
         /** The buffer containing the text node being read */
-        private StringBuilder buffer = new StringBuilder();
+        private final StringBuilder buffer = new StringBuilder();
 
         /** The stack of configuration nodes */
-        private List<ConfigurationNode> stack = new ArrayList<ConfigurationNode>();
+        private final List<ConfigurationNode> stack = new ArrayList<ConfigurationNode>();
 
         public XMLPropertyListHandler(ConfigurationNode root)
         {
@@ -756,7 +760,7 @@ public class XMLPropertyListConfiguratio
         private static final long serialVersionUID = 5586544306664205835L;
 
         /** The list of values in the array. */
-        private List<Object> list = new ArrayList<Object>();
+        private final List<Object> list = new ArrayList<Object>();
 
         /**
          * Add an object to the array.

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/CombinedReloadingController.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/CombinedReloadingController.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/CombinedReloadingController.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/CombinedReloadingController.java Mon Mar 24 21:20:10 2014
@@ -175,6 +175,7 @@ public class CombinedReloadingController
          * controllers. If one of them returns <b>true</b> from its check
          * method, iteration is aborted, and result is <b>true</b>.
          */
+        @Override
         public boolean isReloadingRequired()
         {
             for (ReloadingController rc : owner.getSubControllers())
@@ -191,6 +192,7 @@ public class CombinedReloadingController
          * {@inheritDoc} This implementation resets the reloading state on all
          * managed controllers.
          */
+        @Override
         public void reloadingPerformed()
         {
             for (ReloadingController rc : owner.getSubControllers())

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/FileHandlerReloadingDetector.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/FileHandlerReloadingDetector.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/FileHandlerReloadingDetector.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/FileHandlerReloadingDetector.java Mon Mar 24 21:20:10 2014
@@ -147,6 +147,7 @@ public class FileHandlerReloadingDetecto
      * refresh delay is taken into account, too; a check is only performed if at
      * least this time has passed since the last check.
      */
+    @Override
     public boolean isReloadingRequired()
     {
         long now = System.currentTimeMillis();
@@ -180,6 +181,7 @@ public class FileHandlerReloadingDetecto
      * modification date with the current modification date of the monitored
      * file. So the next change is detected when this file is changed again.
      */
+    @Override
     public void reloadingPerformed()
     {
         updateLastModified(getLastModificationDate());

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/ManagedReloadingDetector.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/ManagedReloadingDetector.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/ManagedReloadingDetector.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/ManagedReloadingDetector.java Mon Mar 24 21:20:10 2014
@@ -31,7 +31,7 @@ public class ManagedReloadingDetector im
         ManagedReloadingDetectorMBean
 {
     /** The logger. */
-    private Log log = LogFactory.getLog(ManagedReloadingDetector.class);
+    private final Log log = LogFactory.getLog(ManagedReloadingDetector.class);
 
     /** A flag whether a reload is required. */
     private volatile boolean reloadingRequired;
@@ -40,6 +40,7 @@ public class ManagedReloadingDetector im
      * {@inheritDoc} This implementation resets the internal flag indicating
      * that a reload should be performed.
      */
+    @Override
     public void reloadingPerformed()
     {
         reloadingRequired = false;
@@ -51,6 +52,7 @@ public class ManagedReloadingDetector im
      *
      * @return a flag whether reloading is required
      */
+    @Override
     public boolean isReloadingRequired()
     {
         return reloadingRequired;
@@ -63,6 +65,7 @@ public class ManagedReloadingDetector im
      *
      * @see org.apache.commons.configuration.reloading.ManagedReloadingDetectorMBean#refresh()
      */
+    @Override
     public void refresh()
     {
         log.info("Reloading configuration.");

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/PeriodicReloadingTrigger.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/PeriodicReloadingTrigger.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/PeriodicReloadingTrigger.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/PeriodicReloadingTrigger.java Mon Mar 24 21:20:10 2014
@@ -212,6 +212,7 @@ public class PeriodicReloadingTrigger
     {
         return new Runnable()
         {
+            @Override
             public void run()
             {
                 controller.checkForReloading(controllerParam);

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/VFSFileHandlerReloadingDetector.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/VFSFileHandlerReloadingDetector.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/VFSFileHandlerReloadingDetector.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/reloading/VFSFileHandlerReloadingDetector.java Mon Mar 24 21:20:10 2014
@@ -51,7 +51,7 @@ import org.apache.commons.vfs2.VFS;
 public class VFSFileHandlerReloadingDetector extends FileHandlerReloadingDetector
 {
     /** Stores the logger.*/
-    private Log log = LogFactory.getLog(getClass());
+    private final Log log = LogFactory.getLog(getClass());
 
     /**
      * Creates a new instance of {@code VFSFileHandlerReloadingDetector} and

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/resolver/CatalogResolver.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/resolver/CatalogResolver.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/resolver/CatalogResolver.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/resolver/CatalogResolver.java Mon Mar 24 21:20:10 2014
@@ -65,7 +65,7 @@ public class CatalogResolver implements 
     /**
      * The CatalogManager
      */
-    private CatalogManager manager = new CatalogManager();
+    private final CatalogManager manager = new CatalogManager();
 
     /**
      * The FileSystem in use.
@@ -173,6 +173,7 @@ public class CatalogResolver implements 
      * @return An InputSource for the mapped identifier, or null.
      * @throws SAXException if an error occurs.
      */
+    @Override
     public InputSource resolveEntity(String publicId, String systemId)
             throws SAXException
     {
@@ -389,7 +390,7 @@ public class CatalogResolver implements 
         private FileSystem fs;
 
         /** FileNameMap to determine the mime type */
-        private FileNameMap fileNameMap = URLConnection.getFileNameMap();
+        private final FileNameMap fileNameMap = URLConnection.getFileNameMap();
 
         /**
          * Load the catalogs.

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/resolver/DefaultEntityResolver.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/resolver/DefaultEntityResolver.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/resolver/DefaultEntityResolver.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/resolver/DefaultEntityResolver.java Mon Mar 24 21:20:10 2014
@@ -38,7 +38,7 @@ import org.xml.sax.SAXException;
 public class DefaultEntityResolver implements EntityResolver, EntityRegistry
 {
     /** Stores a map with the registered public IDs.*/
-    private Map<String, URL> registeredEntities = new HashMap<String, URL>();
+    private final Map<String, URL> registeredEntities = new HashMap<String, URL>();
 
     /**
      * <p>
@@ -62,6 +62,7 @@ public class DefaultEntityResolver imple
      * @param entityURL The URL to use for reading this Entity
      * @throws IllegalArgumentException if the public ID is undefined
      */
+    @Override
     public void registerEntityId(String publicId, URL entityURL)
     {
         if (publicId == null)
@@ -82,6 +83,7 @@ public class DefaultEntityResolver imple
      * @return an input source for the specified entity
      * @throws org.xml.sax.SAXException if a parsing exception occurs
      */
+    @Override
     public InputSource resolveEntity(String publicId, String systemId)
             throws SAXException
     {
@@ -123,6 +125,7 @@ public class DefaultEntityResolver imple
      *
      * @return a map with the registered entity IDs
      */
+    @Override
     public Map<String, URL> getRegisteredEntities()
     {
         return registeredEntities;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/sync/NoOpSynchronizer.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/sync/NoOpSynchronizer.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/sync/NoOpSynchronizer.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/sync/NoOpSynchronizer.java Mon Mar 24 21:20:10 2014
@@ -42,18 +42,22 @@ public enum NoOpSynchronizer implements 
     /** The single shared instance of this class. */
     INSTANCE;
 
+    @Override
     public void beginRead()
     {
     }
 
+    @Override
     public void endRead()
     {
     }
 
+    @Override
     public void beginWrite()
     {
     }
 
+    @Override
     public void endWrite()
     {
     }

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/sync/ReadWriteSynchronizer.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/sync/ReadWriteSynchronizer.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/sync/ReadWriteSynchronizer.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/sync/ReadWriteSynchronizer.java Mon Mar 24 21:20:10 2014
@@ -66,21 +66,25 @@ public class ReadWriteSynchronizer imple
         this(null);
     }
 
+    @Override
     public void beginRead()
     {
         lock.readLock().lock();
     }
 
+    @Override
     public void endRead()
     {
         lock.readLock().unlock();
     }
 
+    @Override
     public void beginWrite()
     {
         lock.writeLock().lock();
     }
 
+    @Override
     public void endWrite()
     {
         lock.writeLock().unlock();

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/ConfigurationNodeVisitorAdapter.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/ConfigurationNodeVisitorAdapter.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/ConfigurationNodeVisitorAdapter.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/ConfigurationNodeVisitorAdapter.java Mon Mar 24 21:20:10 2014
@@ -36,6 +36,7 @@ public class ConfigurationNodeVisitorAda
     /**
      * {@inheritDoc} Empty dummy implementation of this interface method.
      */
+    @Override
     public void visitBeforeChildren(T node, NodeHandler<T> handler)
     {
     }
@@ -43,6 +44,7 @@ public class ConfigurationNodeVisitorAda
     /**
      * {@inheritDoc} Empty dummy implementation of this interface method.
      */
+    @Override
     public void visitAfterChildren(T node, NodeHandler<T> handler)
     {
     }
@@ -51,6 +53,7 @@ public class ConfigurationNodeVisitorAda
      * {@inheritDoc} This implementation returns always <b>false</b>; this means
      * that all nodes in the current hierarchy are traversed.
      */
+    @Override
     public boolean terminate()
     {
         return false;

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/DefaultConfigurationKey.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/DefaultConfigurationKey.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/DefaultConfigurationKey.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/DefaultConfigurationKey.java Mon Mar 24 21:20:10 2014
@@ -625,6 +625,7 @@ public class DefaultConfigurationKey
          *
          * @return a flag if there is a next element
          */
+        @Override
         public boolean hasNext()
         {
             return endIndex < keyBuffer.length();
@@ -635,6 +636,7 @@ public class DefaultConfigurationKey
          *
          * @return the next object
          */
+        @Override
         public Object next()
         {
             return nextKey();
@@ -644,6 +646,7 @@ public class DefaultConfigurationKey
          * Removes the current object in the iteration. This method is not
          * supported by this iterator type, so an exception is thrown.
          */
+        @Override
         public void remove()
         {
             throw new UnsupportedOperationException("Remove not supported!");

Modified: commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/DefaultConfigurationNode.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/DefaultConfigurationNode.java?rev=1581069&r1=1581068&r2=1581069&view=diff
==============================================================================
--- commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/DefaultConfigurationNode.java (original)
+++ commons/proper/configuration/branches/immutableNodes/src/main/java/org/apache/commons/configuration/tree/DefaultConfigurationNode.java Mon Mar 24 21:20:10 2014
@@ -99,6 +99,7 @@ public class DefaultConfigurationNode im
      *
      * @return the name of this node
      */
+    @Override
     public String getName()
     {
         return name;
@@ -109,6 +110,7 @@ public class DefaultConfigurationNode im
      *
      * @param name the new name
      */
+    @Override
     public void setName(String name)
     {
         checkState();
@@ -120,6 +122,7 @@ public class DefaultConfigurationNode im
      *
      * @return the value of this node
      */
+    @Override
     public Object getValue()
     {
         return value;
@@ -130,6 +133,7 @@ public class DefaultConfigurationNode im
      *
      * @param val the value of this node
      */
+    @Override
     public void setValue(Object val)
     {
         value = val;
@@ -140,6 +144,7 @@ public class DefaultConfigurationNode im
      *
      * @return the reference
      */
+    @Override
     public Object getReference()
     {
         return reference;
@@ -150,6 +155,7 @@ public class DefaultConfigurationNode im
      *
      * @param reference the reference object
      */
+    @Override
     public void setReference(Object reference)
     {
         this.reference = reference;
@@ -160,6 +166,7 @@ public class DefaultConfigurationNode im
      *
      * @return the parent node or <b>null </b> if this is the root
      */
+    @Override
     public ConfigurationNode getParentNode()
     {
         return parent;
@@ -170,6 +177,7 @@ public class DefaultConfigurationNode im
      *
      * @param parent the parent of this node
      */
+    @Override
     public void setParentNode(ConfigurationNode parent)
     {
         this.parent = parent;
@@ -180,6 +188,7 @@ public class DefaultConfigurationNode im
      *
      * @param child the new child
      */
+    @Override
     public void addChild(ConfigurationNode child)
     {
         children.addNode(child);
@@ -192,6 +201,7 @@ public class DefaultConfigurationNode im
      *
      * @return a list with all child nodes
      */
+    @Override
     public List<ConfigurationNode> getChildren()
     {
         return children.getSubNodes();
@@ -202,6 +212,7 @@ public class DefaultConfigurationNode im
      *
      * @return the number of all children
      */
+    @Override
     public int getChildrenCount()
     {
         return children.getSubNodes().size();
@@ -213,6 +224,7 @@ public class DefaultConfigurationNode im
      * @param name the name; can be <b>null </b>, then all children are returned
      * @return a list of all children with the given name
      */
+    @Override
     public List<ConfigurationNode> getChildren(String name)
     {
         return children.getSubNodes(name);
@@ -225,6 +237,7 @@ public class DefaultConfigurationNode im
      * children is returned
      * @return the number of child nodes with this name
      */
+    @Override
     public int getChildrenCount(String name)
     {
         return children.getSubNodes(name).size();
@@ -236,6 +249,7 @@ public class DefaultConfigurationNode im
      * @param index the index (0-based)
      * @return the child with this index
      */
+    @Override
     public ConfigurationNode getChild(int index)
     {
         return children.getNode(index);
@@ -247,6 +261,7 @@ public class DefaultConfigurationNode im
      * @param child the node to be removed
      * @return a flag if a node was removed
      */
+    @Override
     public boolean removeChild(ConfigurationNode child)
     {
         return children.removeNode(child);
@@ -258,6 +273,7 @@ public class DefaultConfigurationNode im
      * @param childName the name of the children to be removed
      * @return a flag if at least one child node was removed
      */
+    @Override
     public boolean removeChild(String childName)
     {
         return children.removeNodes(childName);
@@ -266,6 +282,7 @@ public class DefaultConfigurationNode im
     /**
      * Removes all child nodes of this node.
      */
+    @Override
     public void removeChildren()
     {
         children.clear();
@@ -276,6 +293,7 @@ public class DefaultConfigurationNode im
      *
      * @return a flag if this is an attribute node
      */
+    @Override
     public boolean isAttribute()
     {
         return attribute;
@@ -287,6 +305,7 @@ public class DefaultConfigurationNode im
      *
      * @param f the attribute flag
      */
+    @Override
     public void setAttribute(boolean f)
     {
         checkState();
@@ -298,6 +317,7 @@ public class DefaultConfigurationNode im
      *
      * @param attr the attribute to be added
      */
+    @Override
     public void addAttribute(ConfigurationNode attr)
     {
         attributes.addNode(attr);
@@ -311,6 +331,7 @@ public class DefaultConfigurationNode im
      *
      * @return the attribute list, never <b>null </b>
      */
+    @Override
     public List<ConfigurationNode> getAttributes()
     {
         return attributes.getSubNodes();
@@ -321,6 +342,7 @@ public class DefaultConfigurationNode im
      *
      * @return the number of attributes
      */
+    @Override
     public int getAttributeCount()
     {
         return attributes.getSubNodes().size();
@@ -332,6 +354,7 @@ public class DefaultConfigurationNode im
      * @param name the attribute's name
      * @return all attributes with this name
      */
+    @Override
     public List<ConfigurationNode> getAttributes(String name)
     {
         return attributes.getSubNodes(name);
@@ -343,6 +366,7 @@ public class DefaultConfigurationNode im
      * @param name the name
      * @return the number of attributes with this name
      */
+    @Override
     public int getAttributeCount(String name)
     {
         return getAttributes(name).size();
@@ -354,6 +378,7 @@ public class DefaultConfigurationNode im
      * @param node the attribute node to be removed
      * @return a flag if the attribute could be removed
      */
+    @Override
     public boolean removeAttribute(ConfigurationNode node)
     {
         return attributes.removeNode(node);
@@ -365,6 +390,7 @@ public class DefaultConfigurationNode im
      * @param name the name
      * @return a flag if at least one attribute was removed
      */
+    @Override
     public boolean removeAttribute(String name)
     {
         return attributes.removeNodes(name);
@@ -376,6 +402,7 @@ public class DefaultConfigurationNode im
      * @param index the index (0-based)
      * @return the attribute with this index
      */
+    @Override
     public ConfigurationNode getAttribute(int index)
     {
         return attributes.getNode(index);
@@ -384,6 +411,7 @@ public class DefaultConfigurationNode im
     /**
      * Removes all attributes of this node.
      */
+    @Override
     public void removeAttributes()
     {
         attributes.clear();
@@ -395,6 +423,7 @@ public class DefaultConfigurationNode im
      *
      * @return a flag whether this node is defined
      */
+    @Override
     public boolean isDefined()
     {
         return getValue() != null || getChildrenCount() > 0
@@ -406,6 +435,7 @@ public class DefaultConfigurationNode im
      *
      * @param visitor the visitor
      */
+    @Override
     public void visit(ConfigurationNodeVisitor<ConfigurationNode> visitor)
     {
         if (visitor == null)



Mime
View raw message