Re: [JGIT PATCH] RevWalk: Fix RevSort.REVERSE combined with RevSort.TOPO

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Robin Rosenberg <robin.rosenberg.lists@xxxxxxxxxx> wrote:
> söndagen den 7 september 2008 23.00.08 skrev Shawn O. Pearce:
> > From: Shawn O. Pearce <sop@xxxxxxxxxx>
> > 
> > The reverse sort was using the initial pending queue, not the
> > generator that comes before it in the workflow.  This caused
> > a combination of TOPO and REVERSE to return no commits.
> 
> Some unit tests to back this up would be nice. We running low on
> coverage in this section of code.

Yea, I know.  Its way back on my things-to-do list.  Right now I'm
a bit too swamped with day-job to try and put unit tests into the
revwalk package.  There's two open issues on the issue tracker that
tests should be able to cover and permit debugging of.

I'll come back to revwalk tests.  I did some tests for treewalk
not too long ago, but even more are still need there too.  And lets
not even mention the transport package.  ;-)

I ran into this bug because I was relying on RevWalk in my day-job
project.  When it barfed I fixed it, and moved on.
 
> > Signed-off-by: Shawn O. Pearce <sop@xxxxxxxxxx>
> 
> No idea Google employees had this short email addresses.  Ok, 27x27x27 = 19683 
> would be sufficient in theory, I'd suppose.

Yea, you can get a short address if you ask nicely.  Given that
everything is email based its nice having a short address.  Makes it
easy to tell others in person how to reach me.  Fortunately most
of the company uses longer addresses, so three (and four) letter
words are still available.

I must not have my egit.git repository configured right on my
work laptop.  I usually don't do egit work there.

-- 
Shawn.
--
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux