On Thu, Nov 22, 2007 at 01:02:50AM +0100, Wincent Colaiuta wrote: > + return undef if ($#diff == -1); Style nit: I think the rest of the code generally uses (and I prefer) "@diff" to get the number of elements. So: return undef unless @diff; or I might even have written my @diff = ... or return undef; but perhaps I am the only one who finds $#array comparisons to -1 hard on the eyes. -Peff - 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