[Geotools-devel] FindBugz to check code?

[Jody G. <jga...@re...>]

Daniele Romagnoli wrote:
> Hi Christian,
>
> I'm deleting your test resource folder since we shouldn't place any 
> big test data on SVN repos. Mainly for two reasons:
> 1) people having slow connection, will wait a lot when doing a SVN 
> update on the trunk due to download of such a large amount of data.
> 2) svn filesystem deeply grows if committing such data.
>
> Anyway, I have a local copy on my Desk in case you need it back (I 
> will send you it somehow.... just le me know).
>
> Lastly, please dont use CamelCase naming for your module. Just use 
> dashes such as "my-own-module" instead of "myOwnModule"
The section that covered naming conventions when I first joined the 
project was this:
- http://docs.codehaus.org/display/GEOT/5.2+Naming+Conventions

And there was a tool (PMD) that was able to check.

PMD use has died over time; but I find that FindBugz is great. Would it 
be good to add Findbugz to the into the mix in order to catch 
unconventional names; and a range of NPE problems?

At the very least can I recommend FindBugz be run as part of an 
unsupported/module being brought into the library.
Jody