builtin/submodule--helper: factor out submodule updating

diff --git a/diffcore-pickaxe.c b/diffcore-pickaxe.c
index 72bb5a9514f3e806115d30cc669c7ab6855247fb..800a899c8621eaeba7f4246139df0f289ceb6ee2 100644 (file)
--- a/diffcore-pickaxe.c
+++ b/diffcore-pickaxe.c
@@ -215,7 +215,6 @@ static void regcomp_or_die(regex_t *regex, const char *needle, int cflags)
                /* The POSIX.2 people are surely sick */
                char errbuf[1024];
                regerror(err, regex, errbuf, 1024);
-               regfree(regex);
                die("invalid regex: %s", errbuf);
        }
 }
@@ -251,7 +250,6 @@ void diffcore_pickaxe(struct diff_options *o)
                }
        }
 
-       /* Might want to warn when both S and G are on; I don't care... */
        pickaxe(&diff_queued_diff, o, regexp, kws,
                (opts & DIFF_PICKAXE_KIND_G) ? diff_grep : has_changes);