{"thread":{"id":"5159","subject":"EGIT packed delta format reading","startedAt":"2006-08-06T18:37:48Z","lastAt":"2006-08-07T04:07:07Z","messageCount":4,"participants":["Robin Rosenberg","Shawn Pearce"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"24899","messageId":"20060806183748.7591.61536.stgit@lathund.dewire.com","threadId":"5159","inReplyTo":null,"subject":"EGIT packed delta format reading","fromName":"Robin Rosenberg","fromEmail":"robin.rosenberg@dewire.com","sentAt":"2006-08-06T18:37:48Z","receivedAt":"2006-08-06T18:37:48Z","isPatch":false,"sender":{"key":"robin.rosenberg@dewire.com","avatar":"https://avatars.githubusercontent.com/u/46357?v=4"},"body":"Hi,\n\nI deciced to take Shawns Eclipse plugin for a run. There was\nsome problem reading the pack files that seemed to work after\napplying the following fixes, i.e. I could connect to the\nproject and the interal structures looks ok in the debugger\nas far as I can tell, but decorations don't work. That's as\nfar as I've been testing up to now.\n\nBTW. How should EGIT patches be formatted to distingish them\nfrom git patches? I'm sending this using StGit.\n\n-- robin\n"},{"id":"24900","messageId":"20060806184038.7591.12898.stgit@lathund.dewire.com","threadId":"5159","inReplyTo":"20060806183748.7591.61536.stgit@lathund.dewire.com","subject":"[PATCH] Fixes to packed delta format reading.","fromName":"Robin Rosenberg","fromEmail":"robin.rosenberg@dewire.com","sentAt":"2006-08-06T18:40:38Z","receivedAt":"2006-08-06T18:40:38Z","isPatch":true,"sender":{"key":"robin.rosenberg@dewire.com","avatar":"https://avatars.githubusercontent.com/u/46357?v=4"},"body":"From: Robin Rosenberg <robin.rosenberg@dewire.com>\n\n\n---\n\n .../src/org/spearce/jgit/lib/PatchDeltaStream.java |    4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/PatchDeltaStream.java b/org.spearce.jgit/src/org/spearce/jgit/lib/PatchDeltaStream.java\nindex 808e854..11a7679 100644\n--- a/org.spearce.jgit/src/org/spearce/jgit/lib/PatchDeltaStream.java\n+++ b/org.spearce.jgit/src/org/spearce/jgit/lib/PatchDeltaStream.java\n@@ -105,7 +105,7 @@ public class PatchDeltaStream extends In\n                             + \" load delta base for patching.\");\n                     }\n                     shift += n;\n-                    expBaseLen += n;\n+                    expBaseLen -= n;\n                 }\n             }\n             finally\n@@ -269,7 +269,7 @@ public class PatchDeltaStream extends In\n         throws IOException\n     {\n         int r;\n-        while ((r = read(buf, o, len)) > 0)\n+        while ((r = deltaStream.read(buf, o, len)) > 0)\n         {\n             o += r;\n             len -= r;\n"},{"id":"24905","messageId":"20060807040342.GA20514@spearce.org","threadId":"5159","inReplyTo":"20060806184038.7591.12898.stgit@lathund.dewire.com","subject":"Re: [PATCH] Fixes to packed delta format reading.","fromName":"Shawn Pearce","fromEmail":"spearce@spearce.org","sentAt":"2006-08-07T04:03:42Z","receivedAt":"2006-08-07T04:03:42Z","isPatch":true,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"> ---\n> \n>  .../src/org/spearce/jgit/lib/PatchDeltaStream.java |    4 ++--\n>  1 files changed, 2 insertions(+), 2 deletions(-)\n> \n> diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/PatchDeltaStream.java b/org.spearce.jgit/src/org/spearce/jgit/lib/PatchDeltaStream.java\n> index 808e854..11a7679 100644\n> --- a/org.spearce.jgit/src/org/spearce/jgit/lib/PatchDeltaStream.java\n> +++ b/org.spearce.jgit/src/org/spearce/jgit/lib/PatchDeltaStream.java\n> @@ -105,7 +105,7 @@ public class PatchDeltaStream extends In\n>                              + \" load delta base for patching.\");\n>                      }\n>                      shift += n;\n> -                    expBaseLen += n;\n> +                    expBaseLen -= n;\n>                  }\n>              }\n>              finally\n> @@ -269,7 +269,7 @@ public class PatchDeltaStream extends In\n>          throws IOException\n>      {\n>          int r;\n> -        while ((r = read(buf, o, len)) > 0)\n> +        while ((r = deltaStream.read(buf, o, len)) > 0)\n>          {\n>              o += r;\n>              len -= r;\n\nThanks.  I actually found these Friday night when I started reviewing\nthe code to double check its compliance with A Large Angry SCM's\nfile format documentation.  These were already made locally but I\ndidn't push them out to my web repository yet.  I'll do that shortly.\n\n-- \nShawn.\n"},{"id":"24906","messageId":"20060807040707.GB20514@spearce.org","threadId":"5159","inReplyTo":"20060806183748.7591.61536.stgit@lathund.dewire.com","subject":"Re: EGIT packed delta format reading","fromName":"Shawn Pearce","fromEmail":"spearce@spearce.org","sentAt":"2006-08-07T04:07:07Z","receivedAt":"2006-08-07T04:07:07Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Robin Rosenberg <robin.rosenberg@dewire.com> wrote:\n> Hi,\n> \n> I deciced to take Shawns Eclipse plugin for a run. There was\n> some problem reading the pack files that seemed to work after\n> applying the following fixes, i.e. I could connect to the\n> project and the interal structures looks ok in the debugger\n> as far as I can tell, but decorations don't work. That's as\n> far as I've been testing up to now.\n\nI'll take a look at it again, as soon as I get the tree entry\nsorting fixed.\n \n> BTW. How should EGIT patches be formatted to distingish them\n> from git patches? I'm sending this using StGit.\n\nYou can use StGit to send patches, or any other GIT based tool.\nAfter all, we're using GIT.  :-)\n\nAs far as marking patches specifically as an egit/jgit patch you can\nsend them directly to me and CC: the mailing list.  The egit/jgit\nfile names are pretty distinctive, just as the core GIT and Cogito\nfile names are.  I doubt folks will get confused as to which project\na patch belongs to...\n\n-- \nShawn.\n"}]}