Commit cf4030b0 by Carlos Martín Nieto

submodule: remove some obsolete logic

Remove some of the logic that was left-over from the time we had a cache
of submodules, plugging a leak of the submodule object in certain cases.
parent ca2466ff
...@@ -1691,28 +1691,9 @@ static int submodule_load_from_config( ...@@ -1691,28 +1691,9 @@ static int submodule_load_from_config(
/* Deregister under name being replaced */ /* Deregister under name being replaced */
if (replaced) { if (replaced) {
git_submodule_free(sm);
git__free(replaced); git__free(replaced);
} }
/* Insert under alternate key */
if (alternate) {
void *old_sm = NULL;
if (error < 0)
goto done;
if (error > 0)
error = 0;
GIT_REFCOUNT_INC(sm); /* increase refcount for new key */
/* if we replaced an old module under this key, release the old one */
if (old_sm && ((git_submodule *)old_sm) != sm) {
git_submodule_free(old_sm);
/* TODO: log warning about multiple submodules with same path */
}
}
/* TODO: Look up path in index and if it is present but not a GITLINK /* TODO: Look up path in index and if it is present but not a GITLINK
* then this should be deleted (at least to match git's behavior) * then this should be deleted (at least to match git's behavior)
*/ */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment