directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel Lecharny <>
Subject Re: Shared file review
Date Tue, 07 Sep 2010 07:02:35 GMT
  On 9/7/10 6:53 AM, Felix Knecht wrote:
> 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 ...

However, I see that teh file I revewied are way better now :)

Thanks Felix !

Emmanuel L├ęcharny

View raw message