-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 09/07/10 09:02, Emmanuel Lecharny wrote:
On 9/7/10 6:53 AM, Felix Knecht wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
About Checkstyle and Findbugs, I have cleared a lot of items, some of
them will remain, so may be we can tune the config to get rid of them :
'xxx' hide a field (this is because we use the same name in the set(
param ) methods than the associated field.
I tuned the checkstyle configuration [1] and redeployed the reports. For
findbugs I haven't found issues about hidden fields, but maybe I just
overlooked them.
It's on checkstyle, not findbugs.
What is strange is that the warning is still present ...
Of course I could rid of all the 'xxx' hides a field stuff, but I'm not
sure if this is really wanted. If there are some rare special cases
maybe they can be marked as false positives directly either by
annotation or by configuration. But using general configurations (like
"constructor parameters" e.g.) should be the default way to go IMO.
yes. In the Checkstyle doco, we can set this to shut up when we have
parameters hidding a filed for constructors and setters. That should be
enough.
I have modified my local conf in this way.
I also have modified it to remove the warnings for missing Javadocs for
methods which are not public.
--
Regards,
Cordialement,
Emmanuel Lécharny
www.iktek.com
