Thanks David.  I will try to fix as many errors as possible.

It also seems like we do have some findbugs  configuration in build.xml,  I
will see if I could get it working.

On Jan 16, 2017 11:48 AM, "David Smiley" <[email protected]> wrote:

> See https://issues.apache.org/jira/browse/LUCENE-3973  Perhaps you care
> to revive the issue.  I really look forward to static analysis checks.
>
> On Sat, Jan 14, 2017 at 10:19 PM Pushkar Raste <[email protected]>
> wrote:
>
>> Hi,
>> I saw a couple concerning bugs in the code like
>>
>>    - comparing String, Integer, Float and objects of other wrapper types
>>    using == instead of .equals() method.
>>    - Accessing methods on potentially null reference.
>>
>> I ran findbugs on the code base and found a lot of other errors as well.
>> I am working on fixing some of the errors (most of the errors are in Test
>> cases).
>>
>> I was curious, if we run any static code analysis tool as part of build
>> process? More importantly can we force builds to fail if findbugs errors
>> cross a certain threshold.
>>
> --
> Lucene/Solr Search Committer, Consultant, Developer, Author, Speaker
> LinkedIn: http://linkedin.com/in/davidwsmiley | Book: http://www.
> solrenterprisesearchserver.com
>

Reply via email to