accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Josh Elser <>
Subject Re: Checkstyle oddity
Date Tue, 10 Feb 2015 21:30:46 GMT
My thought might be that the module doesn't work right within the @see, 
maybe. The below commit *should* work (as I understand things), but 
fails the checkstyle violation.

Be forewarned, the above branch is based on the pending delegation token 
branch where I saw this.

Christopher wrote:
> I know I tested this rule, and saw it work correctly. Can you send me a
> patch that reproduces the issue you saw?
> --
> Christopher L Tubbs II
> On Mon, Feb 9, 2015 at 10:57 PM, Josh Elser<>  wrote:
>> I had a failure in the checkstyle rules, specifically the unusedimports
>> module.
>> I had a case where Eclipse inserted the import for a class that was only
>> referenced in javadocs. I see that we have the option enabled which should
>> allow this in the checkstyle-plugin configuration, but this doesn't seem to
>> be working as expected.
>> Has anyone else run into this? Is it an upstream checkstyle-plugin bug?
>> I've worked around it by putting the full class name in the javadoc.
>> - Josh

View raw message