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.
https://github.com/joshelser/accumulo/commit/3837db22a288747e068f9fde12618776f35cd107
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
http://gravatar.com/ctubbsii
On Mon, Feb 9, 2015 at 10:57 PM, Josh Elser<[email protected]> 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
