ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jesse Glick <jesse.gl...@sun.com>
Subject Re: svn commit: r475078 - /ant/core/trunk/src/main/org/apache/tools/ant/types/optional/depend/DependScanner.java
Date Wed, 15 Nov 2006 20:46:20 GMT
peterreilly@apache.org wrote:
> inheritDoc
> 
> URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/types/optional/depend/DependScanner.java?view=diff&rev=475078&r1=475077&r2=475078
> ==============================================================================
> -    /**
> -     * @see DirectoryScanner#getIncludedFilesCount
> -     */
> +    /** {@inheritDoc}. */
>      public synchronized int getIncludedFilesCount() {

Note that you can simply delete the Javadoc comment altogether:

/** {@inheritDoc} */

is equivalent to no comment at all, and is appropriate when an override 
of a method adds no notable semantics beyond what is already specified 
in the superclass' Javadoc. Also you probably did not want the trailing 
'.' in there.

>      /**
>       * @see DirectoryScanner#getExcludedDirectories
>       */
> +    /** {@inheritDoc}. */

(and elsewhere)

Did you really mean to leave two comments in place?

-J.

-- 
jesse.glick@sun.com  x22801  netbeans.org  ant.apache.org
       http://google.com/search?q=e%5E%28pi*i%29%2B1


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


Mime
View raw message