On 10/13/06, Rahul Akolkar <[EMAIL PROTECTED]> wrote:
On 10/13/06, Craig McClanahan <[EMAIL PROTECTED]> wrote:
> On 10/13/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
> >
> > Author: rahul
> > Date: Fri Oct 13 07:08:41 2006
> > New Revision: 463684
> >
> > URL: http://svn.apache.org/viewvc?view=rev&rev=463684
> > Log:
> > Unused imports warnings --> organize imports
> > SHALE-310
>
>
> One caution on the "unused imports" warnings ... Checkstyle does not
> consider a javadoc comment like "[EMAIL PROTECTED] Foo}" to be a use, but
Javadoc
> cannot create a correct link unless you either import this class (if it is
> not in the same package), or make it fully qualified. So, some of the
> "unused" imports are actually used :-).
>
<snip/>
Correct, we just discussed this over on commons-dev yesterday (or day
before). But I didn't use the checkstyle report as a baseline, rather
my IDE, which usually is set up to do the right thing (consider @link
bits).
Regardless, I'll go over this commit message tonight and double check.
<snip/>
Done, looks good.
-Rahul
-Rahul