Jay Soffian <jaysoffian@xxxxxxxxx> writes: > - 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, so checking with bottom alone should suffice, no? -- 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