From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vasyl' Vavrychuk Subject: Re: [EGIT PATCH] Fixed trivial warnings. Mainly parametrized raw types, added serialVersionUID, removed unnecessery throws. Date: Mon, 05 Jan 2009 03:08:30 +0200 Message-ID: <49615D8E.9020205@gmail.com> References: <200901050004.41531.robin.rosenberg.lists@dewire.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: "Shawn O. Pearce" , git@vger.kernel.org To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Mon Jan 05 02:10:21 2009 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1LJdz2-0002i0-Fg for gcvg-git-2@gmane.org; Mon, 05 Jan 2009 02:10:20 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751135AbZAEBI7 convert rfc822-to-quoted-printable (ORCPT ); Sun, 4 Jan 2009 20:08:59 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751241AbZAEBI7 (ORCPT ); Sun, 4 Jan 2009 20:08:59 -0500 Received: from main.gmane.org ([80.91.229.2]:44266 "EHLO ciao.gmane.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751029AbZAEBI7 (ORCPT ); Sun, 4 Jan 2009 20:08:59 -0500 Received: from list by ciao.gmane.org with local (Exim 4.43) id 1LJdxg-0005rI-CL for git@vger.kernel.org; Mon, 05 Jan 2009 01:08:56 +0000 Received: from 91.200.115.178 ([91.200.115.178]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 05 Jan 2009 01:08:56 +0000 Received: from vvavrychuk by 91.200.115.178 with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 05 Jan 2009 01:08:56 +0000 X-Injected-Via-Gmane: http://gmane.org/ X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: 91.200.115.178 User-Agent: Thunderbird 2.0.0.19 (Windows/20081209) In-Reply-To: <200901050004.41531.robin.rosenberg.lists@dewire.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Robin Rosenberg wrote: > s=F6ndag 04 januari 2009 23:20:19 skrev Vasyl' Vavrychuk: >> Also fixed: >> 1. "The 'Eclipse-LazyStart' header is deprecated, use 'Bundle-Activa= tionPolicy'" warning. >> 2. Possible NullPointerException warning. >> 3. Unnecessery function parameter warning. >=20 > Thanks for your interest in the projects.=20 > Most changes from a 30 seconds review look reasonable. > However we don't apply changes this way. From your comments it seems = we'd require about > five patches since the changes are of very different types. I thought that commit is trivial. But maybe series of patches will be b= etter because of an ability to revert what we want. >=20 > -- robin >=20 >> diff --git a/org.spearce.egit.core.test/META-INF/MANIFEST.MF b/org.s= pearce.egit.core.test/META-INF/MANIFEST.MF >> index ee5f277..e8bcc79 100644 >> --- a/org.spearce.egit.core.test/META-INF/MANIFEST.MF >> +++ b/org.spearce.egit.core.test/META-INF/MANIFEST.MF >> @@ -11,7 +11,7 @@ Require-Bundle: org.eclipse.core.runtime, >> org.spearce.egit.ui, >> org.spearce.jgit, >> org.eclipse.core.filesystem >> -Eclipse-LazyStart: true >> +Bundle-ActivationPolicy: lazy >=20 > Any pointers on this? (so I can learn) http://help.eclipse.org/ganymede/index.jsp?topic=3D/org.eclipse.platfor= m.doc.isv/reference/misc/bundle_manifest.html The Eclipse-AutoStart and Eclipse-LazyStart headers have been deprecate= d in Eclipse 3.4 >=20 >> diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/interna= l/storage/GitFileHistory.java b/org.spearce.egit.core/src/org/spearce/e= git/core/internal/storage/GitFileHistory.java >> index c01c1c3..61c32ce 100644 >> --- a/org.spearce.egit.core/src/org/spearce/egit/core/internal/stora= ge/GitFileHistory.java >> +++ b/org.spearce.egit.core/src/org/spearce/egit/core/internal/stora= ge/GitFileHistory.java >> @@ -50,11 +50,11 @@ >> GitFileHistory(final IResource rsrc, final int flags, >> final IProgressMonitor monitor) { >> resource =3D rsrc; >> - walk =3D buildWalk(flags); >> + walk =3D buildWalk(/*flags*/); >> revisions =3D buildRevisions(monitor, flags); >> } >> - private KidWalk buildWalk(final int flags) { >> + private KidWalk buildWalk(/*final int flags*/) { > Can't we just drop the parameter completely and wipe every trace of i= t? OK >> final RepositoryMapping rm =3D RepositoryMapping.getMapping(resou= rce); >> if (rm =3D=3D null) { >> Activator.logError("Git not attached to project " >> diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/project= /GitProjectData.java b/org.spearce.egit.core/src/org/spearce/egit/core/= project/GitProjectData.java >> index 04130db..db5f20b 100644 >> --- a/org.spearce.egit.core/src/org/spearce/egit/core/project/GitPro= jectData.java >> +++ b/org.spearce.egit.core/src/org/spearce/egit/core/project/GitPro= jectData.java >> @@ -48,9 +48,9 @@ >> * a Git repository. >> */ >> public class GitProjectData { >> - private static final Map projectDataCache =3D new HashMap(); >> + private static final Map projectDataCach= e =3D new HashMap(); >> =20 >> - private static final Map repositoryCache =3D new HashMap(); >> + private static final Map repositoryCache =3D = new HashMap(); > Been thinking about doing this myself but always found something more= interesting to do. Good. Hope that we will have a generic version of SWT/JFace sometime. Because= now I do not know how to handle an inter operation between generic col= lections and SWT, use SuppressWarning there maybe? >> diff --git a/org.spearce.jgit/src/org/spearce/jgit/treewalk/Canonica= lTreeParser.java b/org.spearce.jgit/src/org/spearce/jgit/treewalk/Canon= icalTreeParser.java >> index dcc53cd..4700510 100644 >> --- a/org.spearce.jgit/src/org/spearce/jgit/treewalk/CanonicalTreePa= rser.java >> +++ b/org.spearce.jgit/src/org/spearce/jgit/treewalk/CanonicalTreePa= rser.java >> @@ -175,8 +175,9 @@ public void back(int delta) { >> // space so this prunes our search more quickly. >> // >> ptr -=3D Constants.OBJECT_ID_LENGTH; >> - while (raw[--ptr] !=3D ' ') >> - /* nothing */; >> + while (raw[--ptr] !=3D ' ') { >> + /* nothing */ >> + } > Not sure if this buys us anything. I wouldn't react if original code = was either way, but > changing it seems unnecessary. With an old version I get "Empty control-flow statement" warning, I've = checked org.eclipse.jdt.core.prefs file, there is: org.eclipse.jdt.core.compiler.problem.emptyStatement=3Dwarning