diff options
| author | Brandon Williams <bmwill@google.com> | 2017-07-25 23:39:15 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-08-02 23:35:08 +0200 |
| commit | 9ef23f91fc22be9327288c8dbb10646bb0bf4340 (patch) | |
| tree | ccad710b035a444190716c38de437c3ce13144cc /submodule.c | |
| parent | t7411: check configuration parsing errors (diff) | |
| download | git-9ef23f91fc22be9327288c8dbb10646bb0bf4340.tar.xz git-9ef23f91fc22be9327288c8dbb10646bb0bf4340.zip | |
submodule: don't use submodule_from_name
The function 'submodule_from_name()' is being used incorrectly here as a
submodule path is being used instead of a submodule name. Since the
correct function to use with a path to a submodule is already being used
('submodule_from_path()') let's remove the call to
'submodule_from_name()'.
Signed-off-by: Brandon Williams <bmwill@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'submodule.c')
| -rw-r--r-- | submodule.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/submodule.c b/submodule.c index 5139b9256b..19bd13bb2a 100644 --- a/submodule.c +++ b/submodule.c @@ -1177,8 +1177,6 @@ static int get_next_submodule(struct child_process *cp, continue; submodule = submodule_from_path(&null_oid, ce->name); - if (!submodule) - submodule = submodule_from_name(&null_oid, ce->name); default_argv = "yes"; if (spf->command_line_option == RECURSE_SUBMODULES_DEFAULT) { |