The handle_builtin() starts from striping of command extension if
STRIP_EXTENSION is enabled. As this is an OS dependent, let's move
to the git-compat-util.h as all similar functions to do handle_builtin()
more cleaner.
---
git-compat-util.h | 18 ++++++++++++++++++
git.c | 13 +------------
2 files changed, 19 insertions(+), 12 deletions(-)
diff --git a/git-compat-util.h b/git-compat-util.h
index 658d03b..57f2fda 100644
--- a/git-compat-util.h
+++ b/git-compat-util.h
@@ -323,6 +323,24 @@ extern char *gitbasename(char *);
#ifndef STRIP_EXTENSION
#define STRIP_EXTENSION ""
+static inline const char *strip_extension(const char **argv)
+{
+ return argv[0];
+}
+#else
+static inline const char *strip_extension(const char **argv)
+{
+ static const char ext[] = STRIP_EXTENSION;
+ int ext_len = strlen(argv[0]) - strlen(ext);
+
+ if (ext_len > 0 && !strcmp(argv[0] + ext_len, ext)) {
+ char *argv0 = xstrdup(argv[0]);
+ argv[0] = argv0;
+ argv0[ext_len] = '\0';
+ }
+
+ return argv[0];
+}
#endif
#ifndef has_dos_drive_prefix
diff --git a/git.c b/git.c
index 8751ef0..a4d2a46 100644
--- a/git.c
+++ b/git.c
@@ -506,19 +506,8 @@ int is_builtin(const char *s)
static void handle_builtin(int argc, const char **argv)
{
- const char *cmd = argv[0];
- int i;
- static const char ext[] = STRIP_EXTENSION;
struct cmd_struct *builtin;
-
- if (sizeof(ext) > 1) {
- i = strlen(argv[0]) - strlen(ext);
- if (i > 0 && !strcmp(argv[0] + i, ext)) {
- char *argv0 = xstrdup(argv[0]);
- argv[0] = cmd = argv0;
- argv0[i] = '\0';
- }
- }
+ const char *cmd = strip_extension(argv);
/* Turn "git cmd --help" into "git help cmd" */
if (argc > 1 && !strcmp(argv[1], "--help")) {
--
2.4.4.764.gf6c74eb.dirty
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
