Re: [PATCH] [submodule] Remove duplicate call to set_rev_name

Heiko Voigt [off-list ref] writes:

Hi,

On Mon, Jun 17, 2013 at 11:55:36AM +0200, Fredrik Gustafsson wrote:

quoted

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.