commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: [Math] Code review
Date Fri, 01 Oct 2010 22:22:50 GMT
On 1 October 2010 22:54, Ted Dunning <ted.dunning@gmail.com> wrote:
> In Mahout, we have found that IntelliJ (which is usually like Eclipse, but
> more thorough) does not warn on the same things
> that findbugs, pmd or checkstyle warn on.  We haven't been able to force
> that behavior very well, either.  That means that
> we have to use a combination of methods to drive down warnings.
>
> In general, the quality of the maven mediated warnings is much lower than
> the quality of the IDE based warnings although

Findbugs (Maven) and Findbugs (IDE) should report the same problems.
However, it's a lot easier to fix the problems when using an IDE.

> there are a few exceptions (Eclipse flags @override all the time even when
> it is correctly done,

I find that Eclipse flags @Override correctly.
However, one does have to ensure that the Eclipse compiler settings are correct.

> findbugs flags certain serialization
> and maintainability problems better).

Agreed.

> On Fri, Oct 1, 2010 at 2:23 PM, Gilles Sadowski <
> gilles@harfang.homelinux.org> wrote:
>
>> This, I know. In fact I was surprised that Sebb uncovered so many problems
>> that were _not_ reported by FindBugs or Checkstyle. Hence my question: Can
>> there be additional Maven plugins that will report all the things which
>> Eclipse sees?
>>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message