On Thu, Jul 28, 2016 at 5:37 PM, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: > On Thu, Jul 28, 2016 at 5:29 PM, Jeff King <peff@xxxxxxxx> wrote: >> >> I guess we want something like: >> >> +void reset_ident_date(void) >> +{ >> + strbuf_reset(&git_default_date); >> +} > > Looks sane. > >> and then to sprinkle calls liberally through builtin-ified programs when >> they move from one unit of work to the next. > > Maybe we can just add it to the end of commit_tree_extended(), and > just say "the cache is reset between commits". > > That way there is no sprinking in random places. Hmph, wouldn't that equally work well if we cleared at the beginning of the function, instead of at the end? -- 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