]> git.ipfire.org Git - thirdparty/git.git/commit
refs.c: remove fallback-to-main-store code get_submodule_ref_store()
authorNguyễn Thái Ngọc Duy <pclouds@gmail.com>
Wed, 23 Aug 2017 12:37:03 +0000 (19:37 +0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 24 Aug 2017 21:59:02 +0000 (14:59 -0700)
commit82a150f27a16cd1425d4c78904470e4b7b8c4cf7
treece6f6406f3f31af9c2898cb2807cb49de9b72c7d
parent32619f99f94760e90c36fd2a4476f459cb11aa41
refs.c: remove fallback-to-main-store code get_submodule_ref_store()

At this state, there are three get_submodule_ref_store() callers:

 - for_each_remote_ref_submodule()
 - handle_revision_pseudo_opt()
 - resolve_gitlink_ref()

The first two deal explicitly with submodules (and we should never fall
back to the main ref store as a result). They are only called from
submodule.c:

 - find_first_merges()
 - submodule_needs_pushing()
 - push_submodule()

The last one, as its name implies, deals only with submodules too, and
the "submodule" (path) argument must be a non-NULL, non-empty string.

So, this "if NULL or empty string" code block should never ever
trigger. And it's wrong to fall back to the main ref store
anyway. Delete it.

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
refs.c