On Feb 10, 2008 3:36 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Marco Costalba <mcostalba@xxxxxxxxx> writes: > > > Currently the --prett=format prefix is looked up in a > > s/prett/&y/; > > > New patch with Junio suggestions included. > > Hmph, this is a blast from the past. > > I do recall pointing out that a rather common "format:%an <%ae>" > ends up parsing the same line twice, and mentioned we may want > to memoise the first call's result in the format_commit_context > structure, but what else did I suggest??? > Please read thread: "[PATCH RESEND] Avoid a useless prefix lookup in strbuf_expand()" you will find your suggestions and following answers. > Anyway, I'll take a look later. Thanks. > Thanks Marco - 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