You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by eb...@apache.org on 2007/04/17 10:52:42 UTC

svn commit: r529531 - in /jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration: AbstractConfiguration.java BaseConfiguration.java CompositeConfiguration.java SubsetConfiguration.java

Author: ebourg
Date: Tue Apr 17 01:52:41 2007
New Revision: 529531

URL: http://svn.apache.org/viewvc?view=rev&rev=529531
Log:
Removed the useless @inheritDoc tags (when no info is added to the javadoc of the super class)
Removed the abstract methods in AbstractConfiguration declared in the Configuration interface and implemented in a subclass

Modified:
    jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/AbstractConfiguration.java
    jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/BaseConfiguration.java
    jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/CompositeConfiguration.java
    jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/SubsetConfiguration.java

Modified: jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/AbstractConfiguration.java
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/AbstractConfiguration.java?view=diff&rev=529531&r1=529530&r2=529531
==============================================================================
--- jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/AbstractConfiguration.java (original)
+++ jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/AbstractConfiguration.java Tue Apr 17 01:52:41 2007
@@ -376,9 +376,6 @@
         });
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public void addProperty(String key, Object value)
     {
         fireEvent(EVENT_ADD_PROPERTY, key, value, true);
@@ -454,27 +451,11 @@
         return base; // just a dummy implementation
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Configuration subset(String prefix)
     {
         return new SubsetConfiguration(this, prefix, ".");
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public abstract boolean isEmpty();
-
-    /**
-     * {@inheritDoc}
-     */
-    public abstract boolean containsKey(String key);
-
-    /**
-     * {@inheritDoc}
-     */
     public void setProperty(String key, Object value)
     {
         fireEvent(EVENT_SET_PROPERTY, key, value, true);
@@ -518,9 +499,6 @@
         // override in sub classes
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public void clear()
     {
         fireEvent(EVENT_CLEAR, null, null, true);
@@ -547,14 +525,6 @@
         fireEvent(EVENT_CLEAR, null, null, false);
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public abstract Iterator getKeys();
-
-    /**
-     * {@inheritDoc}
-     */
     public Iterator getKeys(final String prefix)
     {
         return new FilterIterator(getKeys(), new Predicate()
@@ -567,9 +537,6 @@
         });
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Properties getProperties(String key)
     {
         return getProperties(key, null);
@@ -625,8 +592,6 @@
         return props;
     }
 
-
-
     /**
      * {@inheritDoc}
      * @see PropertyConverter#toBoolean(Object)
@@ -686,9 +651,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public byte getByte(String key)
     {
         Byte b = getByte(key, null);
@@ -702,17 +664,11 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public byte getByte(String key, byte defaultValue)
     {
         return getByte(key, new Byte(defaultValue)).byteValue();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Byte getByte(String key, Byte defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -734,9 +690,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public double getDouble(String key)
     {
         Double d = getDouble(key, null);
@@ -750,17 +703,11 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public double getDouble(String key, double defaultValue)
     {
         return getDouble(key, new Double(defaultValue)).doubleValue();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Double getDouble(String key, Double defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -782,9 +729,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public float getFloat(String key)
     {
         Float f = getFloat(key, null);
@@ -798,17 +742,11 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public float getFloat(String key, float defaultValue)
     {
         return getFloat(key, new Float(defaultValue)).floatValue();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Float getFloat(String key, Float defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -830,9 +768,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public int getInt(String key)
     {
         Integer i = getInteger(key, null);
@@ -846,9 +781,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public int getInt(String key, int defaultValue)
     {
         Integer i = getInteger(key, null);
@@ -861,9 +793,6 @@
         return i.intValue();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Integer getInteger(String key, Integer defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -885,9 +814,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public long getLong(String key)
     {
         Long l = getLong(key, null);
@@ -901,17 +827,11 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public long getLong(String key, long defaultValue)
     {
         return getLong(key, new Long(defaultValue)).longValue();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Long getLong(String key, Long defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -933,9 +853,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public short getShort(String key)
     {
         Short s = getShort(key, null);
@@ -949,17 +866,11 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public short getShort(String key, short defaultValue)
     {
         return getShort(key, new Short(defaultValue)).shortValue();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Short getShort(String key, Short defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -1002,9 +913,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public BigDecimal getBigDecimal(String key, BigDecimal defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -1047,9 +955,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public BigInteger getBigInteger(String key, BigInteger defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -1092,9 +997,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public String getString(String key, String defaultValue)
     {
         Object value = resolveContainerStore(key);
@@ -1174,9 +1076,6 @@
         return getList(key, new ArrayList());
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public List getList(String key, List defaultValue)
     {
         Object value = getProperty(key);
@@ -1198,7 +1097,6 @@
             {
                 list.add(interpolate(it.next()));
             }
-
         }
         else if (value == null)
         {

Modified: jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/BaseConfiguration.java
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/BaseConfiguration.java?view=diff&rev=529531&r1=529530&r2=529531
==============================================================================
--- jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/BaseConfiguration.java (original)
+++ jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/BaseConfiguration.java Tue Apr 17 01:52:41 2007
@@ -133,9 +133,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public void clear()
     {
         fireEvent(EVENT_CLEAR, null, null, true);

Modified: jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/CompositeConfiguration.java
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/CompositeConfiguration.java?view=diff&rev=529531&r1=529530&r2=529531
==============================================================================
--- jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/CompositeConfiguration.java (original)
+++ jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/CompositeConfiguration.java Tue Apr 17 01:52:41 2007
@@ -204,9 +204,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Iterator getKeys()
     {
         List keys = new ArrayList();
@@ -228,9 +225,6 @@
         return keys.iterator();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Iterator getKeys(String key)
     {
         List keys = new ArrayList();
@@ -252,9 +246,6 @@
         return keys.iterator();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public boolean isEmpty()
     {
         boolean isEmpty = true;
@@ -270,9 +261,6 @@
         return isEmpty;
     }
 
-    /**
-     * {@inheritDoc}
-     */
     protected void clearPropertyDirect(String key)
     {
         for (Iterator i = configList.iterator(); i.hasNext();)
@@ -282,9 +270,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public boolean containsKey(String key)
     {
         for (Iterator i = configList.iterator(); i.hasNext();)
@@ -298,9 +283,6 @@
         return false;
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public List getList(String key, List defaultValue)
     {
         List list = new ArrayList();
@@ -333,9 +315,6 @@
         return list;
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public String[] getStringArray(String key)
     {
         List list = getList(key);

Modified: jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/SubsetConfiguration.java
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/SubsetConfiguration.java?view=diff&rev=529531&r1=529530&r2=529531
==============================================================================
--- jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/SubsetConfiguration.java (original)
+++ jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/SubsetConfiguration.java Tue Apr 17 01:52:41 2007
@@ -149,65 +149,41 @@
         this.prefix = prefix;
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Configuration subset(String prefix)
     {
         return parent.subset(getParentKey(prefix));
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public boolean isEmpty()
     {
         return !getKeys().hasNext();
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public boolean containsKey(String key)
     {
         return parent.containsKey(getParentKey(key));
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public void addPropertyDirect(String key, Object value)
     {
         parent.addProperty(getParentKey(key), value);
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public void setProperty(String key, Object value)
     {
         parent.setProperty(getParentKey(key), value);
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public void clearProperty(String key)
     {
         parent.clearProperty(getParentKey(key));
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Object getProperty(String key)
     {
         return parent.getProperty(getParentKey(key));
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Iterator getKeys(String prefix)
     {
         return new TransformIterator(parent.getKeys(getParentKey(prefix)), new Transformer()
@@ -219,9 +195,6 @@
         });
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public Iterator getKeys()
     {
         return new TransformIterator(parent.getKeys(prefix), new Transformer()
@@ -233,9 +206,6 @@
         });
     }
 
-    /**
-     * {@inheritDoc}
-     */
     protected Object interpolate(Object base)
     {
         if (delimiter == null && "".equals(prefix))
@@ -249,9 +219,6 @@
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
     protected String interpolate(String base)
     {
         return super.interpolate(base);



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