{"thread":{"id":"15413","subject":"[JGIT PATCH] RevWalk: Fix RevSort.REVERSE combined with RevSort.TOPO","startedAt":"2008-09-07T21:00:08Z","lastAt":"2008-09-07T21:32:27Z","messageCount":4,"participants":["Shawn O. Pearce","Robin Rosenberg"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"90006","messageId":"1220821208-13420-1-git-send-email-spearce@spearce.org","threadId":"15413","inReplyTo":null,"subject":"[JGIT PATCH] RevWalk: Fix RevSort.REVERSE combined with RevSort.TOPO","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-09-07T21:00:08Z","receivedAt":"2008-09-07T21:00:08Z","isPatch":true,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"From: Shawn O. Pearce \n\nThe reverse sort was using the initial pending queue, not the\ngenerator that comes before it in the workflow. This caused\na combination of TOPO and REVERSE to return no commits.\n\nSigned-off-by: Shawn O. Pearce \n---\n .../org/spearce/jgit/revwalk/StartGenerator.java | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/org.spearce.jgit/src/org/spearce/jgit/revwalk/StartGenerator.java b/org.spearce.jgit/src/org/spearce/jgit/revwalk/StartGenerator.java\nindex 7ddcd3c..1b7947f 100644\n--- a/org.spearce.jgit/src/org/spearce/jgit/revwalk/StartGenerator.java\n+++ b/org.spearce.jgit/src/org/spearce/jgit/revwalk/StartGenerator.java\n@@ -144,7 +144,7 @@ RevCommit next() throws MissingObjectException,\n \t\t\t\t&& (g.outputType() & SORT_TOPO) == 0)\n \t\t\tg = new TopoSortGenerator(g);\n \t\tif (walker.hasRevSort(RevSort.REVERSE))\n-\t\t\tg = new LIFORevQueue(q);\n+\t\t\tg = new LIFORevQueue(g);\n \t\tif (boundary)\n \t\t\tg = new BoundaryGenerator(w, g);\n \n-- \n1.6.0.1.319.g9f32b\n"},{"id":"90007","messageId":"20080907210308.GA15460@spearce.org","threadId":"15413","inReplyTo":"1220821208-13420-1-git-send-email-spearce@spearce.org","subject":"Re: [JGIT PATCH] RevWalk: Fix RevSort.REVERSE combined with RevSort.TOPO","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-09-07T21:03:08Z","receivedAt":"2008-09-07T21:03:08Z","isPatch":true,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"\"Shawn O. Pearce\" wrote:\n> From: Shawn O. Pearce \n> \n> The reverse sort was using the initial pending queue, not the\n> generator that comes before it in the workflow. This caused\n> a combination of TOPO and REVERSE to return no commits.\n> \n> Signed-off-by: Shawn O. Pearce \n\nArrrgh. Wrong email.\n\nDrop the From and fix the SOB to be:\n\nSigned-off-by: Shawn O. Pearce \n\n-- \nShawn.\n"},{"id":"90009","messageId":"200809072321.59789.robin.rosenberg.lists@dewire.com","threadId":"15413","inReplyTo":"1220821208-13420-1-git-send-email-spearce@spearce.org","subject":"Re: [JGIT PATCH] RevWalk: Fix RevSort.REVERSE combined with RevSort.TOPO","fromName":"Robin Rosenberg","fromEmail":"robin.rosenberg.lists@dewire.com","sentAt":"2008-09-07T21:21:59Z","receivedAt":"2008-09-07T21:21:59Z","isPatch":true,"sender":{"key":"robin.rosenberg@dewire.com","avatar":"https://avatars.githubusercontent.com/u/46357?v=4"},"body":"söndagen den 7 september 2008 23.00.08 skrev Shawn O. Pearce:\n> From: Shawn O. Pearce \n> \n> The reverse sort was using the initial pending queue, not the\n> generator that comes before it in the workflow. This caused\n> a combination of TOPO and REVERSE to return no commits.\n\nSome unit tests to back this up would be nice. We running low on\ncoverage in this section of code.\n\n> Signed-off-by: Shawn O. Pearce \n\nNo idea Google employees had this short email addresses. Ok, 27x27x27 = 19683 \nwould be sufficient in theory, I'd suppose.\n\n-- robin\n"},{"id":"90010","messageId":"20080907213227.GB15460@spearce.org","threadId":"15413","inReplyTo":"200809072321.59789.robin.rosenberg.lists@dewire.com","subject":"Re: [JGIT PATCH] RevWalk: Fix RevSort.REVERSE combined with RevSort.TOPO","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-09-07T21:32:27Z","receivedAt":"2008-09-07T21:32:27Z","isPatch":true,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Robin Rosenberg wrote:\n> söndagen den 7 september 2008 23.00.08 skrev Shawn O. Pearce:\n> > From: Shawn O. Pearce \n> > \n> > The reverse sort was using the initial pending queue, not the\n> > generator that comes before it in the workflow. This caused\n> > a combination of TOPO and REVERSE to return no commits.\n> \n> Some unit tests to back this up would be nice. We running low on\n> coverage in this section of code.\n\nYea, I know. Its way back on my things-to-do list. Right now I'm\na bit too swamped with day-job to try and put unit tests into the\nrevwalk package. There's two open issues on the issue tracker that\ntests should be able to cover and permit debugging of.\n\nI'll come back to revwalk tests. I did some tests for treewalk\nnot too long ago, but even more are still need there too. And lets\nnot even mention the transport package. ;-)\n\nI ran into this bug because I was relying on RevWalk in my day-job\nproject. When it barfed I fixed it, and moved on.\n \n> > Signed-off-by: Shawn O. Pearce \n> \n> No idea Google employees had this short email addresses. Ok, 27x27x27 = 19683 \n> would be sufficient in theory, I'd suppose.\n\nYea, you can get a short address if you ask nicely. Given that\neverything is email based its nice having a short address. Makes it\neasy to tell others in person how to reach me. Fortunately most\nof the company uses longer addresses, so three (and four) letter\nwords are still available.\n\nI must not have my egit.git repository configured right on my\nwork laptop. I usually don't do egit work there.\n\n-- \nShawn.\n"}]}