Return-Path: X-Original-To: apmail-lucene-dev-archive@www.apache.org Delivered-To: apmail-lucene-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 1FF256A7F for ; Fri, 8 Jul 2011 17:16:47 +0000 (UTC) Received: (qmail 94484 invoked by uid 500); 8 Jul 2011 17:16:45 -0000 Delivered-To: apmail-lucene-dev-archive@lucene.apache.org Received: (qmail 94168 invoked by uid 500); 8 Jul 2011 17:16:45 -0000 Mailing-List: contact dev-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@lucene.apache.org Delivered-To: mailing list dev@lucene.apache.org Received: (qmail 94161 invoked by uid 99); 8 Jul 2011 17:16:44 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 08 Jul 2011 17:16:44 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=5.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: local policy) Received: from [188.138.97.18] (HELO serv1.sd-datasolutions.de) (188.138.97.18) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 08 Jul 2011 17:16:39 +0000 Received: from VEGA (port-92-196-55-61.dynamic.qsc.de [92.196.55.61]) by serv1.sd-datasolutions.de (Postfix) with ESMTPSA id 0528D14AA41E for ; Fri, 8 Jul 2011 17:16:17 +0000 (UTC) From: "Uwe Schindler" To: References: <673ED01A-D2F3-4516-B5F6-633E202BEFF7@mitre.org> <9B3E4AE0-2D13-407B-83F9-69834B0D052A@mitre.org> In-Reply-To: <9B3E4AE0-2D13-407B-83F9-69834B0D052A@mitre.org> Subject: RE: FindBugs & PMD ? Date: Fri, 8 Jul 2011 19:16:19 +0200 Message-ID: <001b01cc3d92$c13e0ad0$43ba2070$@thetaphi.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Outlook 14.0 Thread-Index: AQGRW+tVd/Pn7SOpMGaokd8KF04i3gDmK2/aA0TLB1OVNxWngA== Content-Language: de Just a stupid question: Once you add those annotations, wouldn't the JAR file not require then this annotations.jar? Or are all of them not available to runtime? Uwe ----- Uwe Schindler H.-H.-Meier-Allee 63, D-28213 Bremen http://www.thetaphi.de eMail: uwe@thetaphi.de > -----Original Message----- > From: Smiley, David W. [mailto:dsmiley@mitre.org] > Sent: Friday, July 08, 2011 4:30 PM > To: dev@lucene.apache.org > Subject: Re: FindBugs & PMD ? > > Rob, there is an ASL 2.0 licensed implementation here: > https://github.com/stephenc/findbugs-annotations > > ~ David > > On Jul 8, 2011, at 10:12 AM, Robert Muir wrote: > > > On Fri, Jul 8, 2011 at 10:08 AM, Smiley, David W. > wrote: > >> Developers, > >> Any thoughts on using FindBugs & PMD to catch more bugs in > Lucene/Solr? Jenkins could be configured to run FindBugs & PMD analysis > nightly. It would have helped find this: > >> > >> (LUCENE-3294) Some code still compares string equality > >> instead using equals > >> > >> I am aware there are a high degree of false-positives but there are ways > of dealing with them, such as with @SuppressWarnings("PMD") and with > //NOPMD and for Findbugs, there is > @edu.umd.cs.findbugs.annotations.SuppressWarnings(....) and there's a > fairly detailed configuration file for FindBugs to really control it and to make > exceptions. I'd also really like to see use of FindBugs concurrency > annotations @GuardedBy, @Immutable, @NotThreadSafe, @ThreadSafe. > > > > I think its a good idea for nightly, but I am strongly against linking > > to an LGPL library for these annotations. > > I would prefer PMD instead, because of the license. > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org For > > additional commands, e-mail: dev-help@lucene.apache.org > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org For additional > commands, e-mail: dev-help@lucene.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org For additional commands, e-mail: dev-help@lucene.apache.org