Implementation was already static, it's just a fix for clarity and potential speed-up. Reported-by: Shawn O. Pearce <spearce@xxxxxxxxxxx> Signed-off-by: Marek Zawirski <marek.zawirski@xxxxxxxxx> --- It could be squashed with patch 19/20. I can clean up this mess, adding also Shawn's patch - just let me know what is preferred way (squash commits, commits on top?). .../src/org/spearce/jgit/lib/PackWriter.java | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/PackWriter.java b/org.spearce.jgit/src/org/spearce/jgit/lib/PackWriter.java index 18d3ec2..ba43da5 100644 --- a/org.spearce.jgit/src/org/spearce/jgit/lib/PackWriter.java +++ b/org.spearce.jgit/src/org/spearce/jgit/lib/PackWriter.java @@ -543,8 +543,9 @@ public class PackWriter { } } - private boolean isBetterDeltaReuseLoader(PackedObjectLoader currentLoader, - PackedObjectLoader loader) throws IOException { + private static boolean isBetterDeltaReuseLoader( + PackedObjectLoader currentLoader, PackedObjectLoader loader) + throws IOException { if (currentLoader == null) return true; if (loader.getRawSize() < currentLoader.getRawSize()) -- 1.5.5.1 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html