Re: [PATCH] blame: prevent a segv when -L given start > EOF

On Mon, Feb 8, 2010 at 10:55 PM, Junio C Hamano [off-list ref] wrote:

Jay Soffian [off-list ref] writes:

quoted

-     if (lno < top)
+     if (lno < top || lno < bottom)
              die("file %s has only %lu lines", path, lno);

Thanks; I think we make sure that "bottom < top" always hold true before
we reach this point,

We swap them if they're reversed.

so checking with bottom alone should suffice, no?

Ah true. You can squash that in? :-)

j.