Jeff King <peff@xxxxxxxx> writes: > On Sun, Feb 21, 2016 at 11:41:35PM +0000, John Keeping wrote: > >> My original sed version was: >> >> sed -ne "/^author /p" -e "/^summary /p" >> >> which I think will work on all platforms (we already use it in >> t0000-basic.sh) but then I decided to be too clever :-( >> >> I still think sed is simpler than introducing a new function to wrap a >> perl script. > > Yeah, I think that is good (personally I'd use a function anyway, but I > think it is short enough that we could go either way). Agreed, and because there are repeated invocation of the same sed script in this file, it would be sensible to hide it behind a helper function. Thanks. -- 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