Re: [Patch] DirectoryScanner

[Kev Jackson]

    /**
-     * gets whether or not a DirectoryScanner
follows symbolic links
+     * Sets whether or not a DirectoryScanner
follows symbolic links
     *
-     * @return flag indicating whether symbolic
links should be followed
+     * @return whether or not symbolic links should
be followed
     *
     * @since ant 1.6
     */
   

[SNIP]
Is that the doc for isFollowSymlinks() that you have
changed to "Sets whether..."?
 

Yes, sorry typo, but a very bad one at that since it is the complete 
opposite of the code.

I wonder if there is a javadoc tool that will check the semantic meaning 
of comments against code - similar to checkstyle but comparing method 
code vs doc comments.  That would catch these mistakes without hassling 
overworked commiters :)

Sorry again, I'll try to be more careful with anything else that I 
submit. /me suitably chastised.

Kev

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]