Re: [PATCH] stash: fix false positive in the invalid ref test.

[Jeff King <peff@github.com>]

On Thu, Apr 07, 2011 at 08:47:07AM +1000, Jon Seymour wrote:

> On Thu, Apr 7, 2011 at 4:27 AM, Jeff King <peff@github.com> wrote:
> > On Wed, Apr 06, 2011 at 09:21:13AM +1000, Jon Seymour wrote:
>     git stash drop
> >
> > Probably we should just squash your fix in with my first patch, and drop
> > my test.  Your fixed version is a superset of what mine tests.
> >
> 
> I was wondering if it might be better to break up the original test,
> so that each sub-function has its own invalid ref test? If there is
> agreement, I can do this on top of 9355fc which is the tip of the
> branch containing these fixes that has been merged into pu.

I don't think it's that big a deal. You could do a more thorough test
for each, I guess (to make sure they not just failed but also show that
they didn't have any effects), but that would not be checking an actual
regression we have seen. IOW, I'm sure you have something more
productive to do with your time. :)

-Peff