Heya, On Thu, Aug 4, 2011 at 21:34, Junio C Hamano <gitster@xxxxxxxxx> wrote: > has_action () { > test -s "$1" > } > has_action () { > sane_grep -v -e '^#' -e '^[ ]*$' "$1" >/dev/null > } I think the former more correctly checks what the function name implies, is there any downside to that which makes you suggest this second approach? -- Cheers, Sverre Rabbelier -- 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