From 85a2a99cf6d461a7c5ee67aab31331fbf374935c Mon Sep 17 00:00:00 2001 From: Robin Rosenberg Date: Tue, 5 Feb 2008 00:49:22 +0100 Subject: [PATCH] Cleanup unboxing/boxing These operation are relatively expensive in general so it is good to make them visible, but when they are needed a lot we just want to do without explicit conversion so we ignore the warning there. Signed-off-by: Robin Rosenberg --- .../src/org/spearce/egit/ui/GitHistoryPage.java | 1 + .../egit/ui/internal/decorators/GitResourceDecorator.java | 10 +++++----- org.spearce.jgit.test/tst/org/spearce/jgit/lib/LaneTest.java | 1 + .../tst/org/spearce/jgit/lib/SuperListTest.java | 1 + org.spearce.jgit/src/org/spearce/jgit/lib/PersonIdent.java | 2 +- 5 files changed, 9 insertions(+), 6 deletions(-) diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/GitHistoryPage.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/GitHistoryPage.java index 812747d3..2fc7afa5 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/GitHistoryPage.java +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/GitHistoryPage.java @@ -864,6 +864,7 @@ public class GitHistoryPage extends HistoryPage implements IAdaptable, // SWT.COLOR_WHITE, SWT.COLOR_YELLOW }; + @SuppressWarnings("boxing") public void handleEvent(Event event) { TableItem item = (TableItem) event.item; if (event.index == 0) { diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/decorators/GitResourceDecorator.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/decorators/GitResourceDecorator.java index c13c38ac..7eb008ba 100644 --- a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/decorators/GitResourceDecorator.java +++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/decorators/GitResourceDecorator.java @@ -207,7 +207,7 @@ public class GitResourceDecorator extends LabelProvider implements return Boolean.FALSE; } - return mapped.isResourceChanged(rsrc); + return new Boolean(mapped.isResourceChanged(rsrc)); } return null; // not mapped } catch (CoreException e) { @@ -244,8 +244,8 @@ public class GitResourceDecorator extends LabelProvider implements if (rsrc instanceof IContainer) { Integer df = (Integer) rsrc .getSessionProperty(GITFOLDERDIRTYSTATEPROPERTY); - Boolean f = df == null ? isDirty(rsrc) : df - .intValue() == CHANGED; + Boolean f = df == null ? isDirty(rsrc) + : new Boolean(df.intValue() == CHANGED); if (f != null) { if (f.booleanValue()) { decoration.addPrefix(">"); // Have not @@ -358,7 +358,7 @@ public class GitResourceDecorator extends LabelProvider implements try { Integer dirty = (Integer) rsrc.getSessionProperty(GITFOLDERDIRTYSTATEPROPERTY); if (dirty == null) { - rsrc.setSessionProperty(GITFOLDERDIRTYSTATEPROPERTY, flag); + rsrc.setSessionProperty(GITFOLDERDIRTYSTATEPROPERTY, new Integer(flag)); Activator.trace("SETTING:"+rsrc.getFullPath().toOSString()+" => "+flag); orState(rsrc.getParent(), flag); Display.getDefault().asyncExec(new Runnable() { @@ -375,7 +375,7 @@ public class GitResourceDecorator extends LabelProvider implements }); } else { if ((dirty.intValue() | flag) != dirty.intValue()) { - dirty = dirty | flag; + dirty = new Integer(dirty.intValue() | flag); rsrc.setSessionProperty(GITFOLDERDIRTYSTATEPROPERTY, dirty); Activator.trace("SETTING:"+rsrc.getFullPath().toOSString()+" => "+dirty); orState(rsrc.getParent(), dirty.intValue()); diff --git a/org.spearce.jgit.test/tst/org/spearce/jgit/lib/LaneTest.java b/org.spearce.jgit.test/tst/org/spearce/jgit/lib/LaneTest.java index 0f57075f..707fb866 100644 --- a/org.spearce.jgit.test/tst/org/spearce/jgit/lib/LaneTest.java +++ b/org.spearce.jgit.test/tst/org/spearce/jgit/lib/LaneTest.java @@ -498,6 +498,7 @@ public class LaneTest extends TestCase { , road); } + @SuppressWarnings("boxing") private String drawAsAscii(TopologicalSorter counter) { StringWriter w = new StringWriter(); List entries = counter.getEntries(); diff --git a/org.spearce.jgit.test/tst/org/spearce/jgit/lib/SuperListTest.java b/org.spearce.jgit.test/tst/org/spearce/jgit/lib/SuperListTest.java index 121bf9ca..79f4da65 100644 --- a/org.spearce.jgit.test/tst/org/spearce/jgit/lib/SuperListTest.java +++ b/org.spearce.jgit.test/tst/org/spearce/jgit/lib/SuperListTest.java @@ -28,6 +28,7 @@ public class SuperListTest extends TestCase { assertEquals(0, l.size()); } + @SuppressWarnings("boxing") public void testNonEmpty() { List l = new SuperList(); List sl1 = Arrays.asList(new Integer[] { 3,4 }); diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/PersonIdent.java b/org.spearce.jgit/src/org/spearce/jgit/lib/PersonIdent.java index 5857cb77..6879eb74 100644 --- a/org.spearce.jgit/src/org/spearce/jgit/lib/PersonIdent.java +++ b/org.spearce.jgit/src/org/spearce/jgit/lib/PersonIdent.java @@ -62,7 +62,7 @@ public class PersonIdent { name = username; emailAddress = email; - when = Calendar.getInstance().getTimeInMillis(); + when = new Long(Calendar.getInstance().getTimeInMillis()); tzOffset = TimeZone.getDefault().getOffset(when.longValue()) / (60 * 1000); } -- 2.11.4.GIT