Re: Any Checkstyle users?



Rick Giles wrote:

[me:]
> > The real problem here is not the use of a "magic number", but the use
> > of a checking tool which will blindly apply the same so-called rule to
> > every situation, regardless of whether it is applicable. If Checkstyle
> > doesn't provide a way to annotate your code to say "I have reviewed
> > this 'problem' code and it is fine as it is", then I'd suggest dropping
> > Checkstyle.
> >
> Yes, it does; see SuppressionCommentFilter at
> http://checkstyle.sourceforge.net/config.html

Good. (Although the feature seems rather awkward to use for my taste.)

BTW, it appears (from more recent posts in the thread) that the issue wasn't
the use of a magic number at all, but of a variable of type HashMap.

-- chris



.