Heiko Voigt <hvoigt@xxxxxxxxxx> writes: > Hi, > > On Mon, Jun 17, 2013 at 11:55:36AM +0200, Fredrik Gustafsson wrote: >> set_rev_name is a possible expensive operation. If a submodule has >> changes in it, set_rev_name was called twice. >> >> Solution is to move set_rev_name so it's only called once, no matter the >> codepath taken. > > Looks good to me. 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