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. Cheers Heiko -- 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