Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (nike.apache.org: domain of thomas.neidhart@gmail.com
 designates 209.85.212.177 as permitted sender)
Message-ID: <50202BF6.2090507@gmail.com>
Date: Mon, 06 Aug 2012 22:41:26 +0200
From: Thomas Neidhart <thomas.neidhart@gmail.com>
User-Agent: Mozilla/5.0 (X11; Linux i686;
 rv:14.0) Gecko/20120714 Thunderbird/14.0
MIME-Version: 1.0
To: dev@commons.apache.org
Subject: Re: Fwd: svn commit: r1369931 - in
 /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections:
 keyvalue/ list/
References: <3710293.FeXhuDX6Eq@floh>
In-Reply-To: <3710293.FeXhuDX6Eq@floh>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit

On 08/06/2012 10:00 PM, J�rg Schaible wrote:
> 
> =================== %< ==========================
> 
> Betreff: svn commit: r1369931 - in 
> /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections: 
> keyvalue/ list/
> Absender: tn-1oDqGaOF3Lkdnm+yROfE0A@public.gmane.org
> Datum: Mon, 06 Aug 2012 19:21:30 +0000
> Newsgruppe: gmane.comp.jakarta.commons.scm
> 
> Author: tn
> Date: Mon Aug  6 19:21:29 2012
> New Revision: 1369931
> 
> URL: http://svn.apache.org/viewvc?rev=1369931&view=rev
> Log:
> Checkstyle fixes.
> 
> [snip]
> 
> @@ -55,14 +55,18 @@ public abstract class AbstractMapEntryDe
>      }
>  
>      //-----------------------------------------------------------------------
> +    
> +    /** {@inheritDoc} */
>      public K getKey() {
>          return entry.getKey();
>      }
>  
> +    /** {@inheritDoc} */
>      public V getValue() {
>          return entry.getValue();
>      }
>  
> +    /** {@inheritDoc} */
>      public V setValue(V object) {
>          return entry.setValue(object);
>      }
> =================== %< ==========================
> 
> Geeez, what's that for an annoying change? Since Java 5 this is already the 
> default for Javadoc when overwriting/implementing methods, so adding a Javdoc 
> comment with a single @inheritDoc is completely superfluous and adds simply 
> clutter!
> 
> Can we stop Checkstyle complaining about it and revert these lines again?

If you have a better solution, I am really willing to include it.

My suggestion to go for java 6 source compatibility to be able to use
@Override tags was so far objected.

Thomas

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