]> granicus.if.org Git - cgit/commitdiff
cmd: no need for pre function hook now
authorJason A. Donenfeld <Jason@zx2c4.com>
Fri, 14 Aug 2015 13:54:32 +0000 (15:54 +0200)
committerJason A. Donenfeld <Jason@zx2c4.com>
Fri, 14 Aug 2015 13:54:32 +0000 (15:54 +0200)
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
cgit.c
cmd.c
cmd.h

diff --git a/cgit.c b/cgit.c
index 727141232222310d0ad29eaf0797609c8953dc16..28a2f149c6cd41d8c113acf4d6e9db254a82685f 100644 (file)
--- a/cgit.c
+++ b/cgit.c
@@ -733,9 +733,6 @@ static void process_request(void)
        if (ctx.repo && prepare_repo_cmd())
                return;
 
-       if (cmd->pre)
-               cmd->pre();
-
        cmd->fn();
 }
 
diff --git a/cmd.c b/cmd.c
index 05494fe145b92b030a65d2290c0c474368d00b08..20c80b0d6d5f2b3b531f8b70353f0293fa16006d 100644 (file)
--- a/cmd.c
+++ b/cmd.c
@@ -38,21 +38,17 @@ static void atom_fn(void)
 
 static void about_fn(void)
 {
-       if (ctx.repo)
-               cgit_print_repo_readme(ctx.qry.path);
-       else
+       if (ctx.repo) {
+               if (!ctx.qry.path &&
+                   ctx.qry.url[strlen(ctx.qry.url) - 1] != '/' &&
+                   ctx.env.path_info[strlen(ctx.env.path_info) - 1] != '/')
+                       cgit_redirect(fmtalloc("%s/", cgit_currenturl()), true);
+               else
+                       cgit_print_repo_readme(ctx.qry.path);
+       } else
                cgit_print_site_readme();
 }
 
-static void about_pre(void)
-{
-       if (ctx.repo &&
-           !ctx.qry.path &&
-           ctx.qry.url[strlen(ctx.qry.url) - 1] != '/' &&
-           ctx.env.path_info[strlen(ctx.env.path_info) - 1] != '/')
-               cgit_redirect(fmtalloc("%s/", cgit_currenturl()), true);
-}
-
 static void blob_fn(void)
 {
        cgit_print_blob(ctx.qry.sha1, ctx.qry.path, ctx.qry.head, 0);
@@ -145,8 +141,6 @@ static void tree_fn(void)
        cgit_print_tree(ctx.qry.sha1, ctx.qry.path);
 }
 
-#define def_cmp(name, want_repo, want_vpath, is_clone) \
-       {#name, name##_fn, name##_pre, want_repo, want_vpath, is_clone}
 #define def_cmd(name, want_repo, want_vpath, is_clone) \
        {#name, name##_fn, NULL, want_repo, want_vpath, is_clone}
 
@@ -155,7 +149,7 @@ struct cgit_cmd *cgit_get_cmd(void)
        static struct cgit_cmd cmds[] = {
                def_cmd(HEAD, 1, 0, 1),
                def_cmd(atom, 1, 0, 0),
-               def_cmp(about, 0, 0, 0),
+               def_cmd(about, 0, 0, 0),
                def_cmd(blob, 1, 0, 0),
                def_cmd(commit, 1, 1, 0),
                def_cmd(diff, 1, 1, 0),
diff --git a/cmd.h b/cmd.h
index 1a9808950b4be77732eb764ff519da237945dee5..6249b1d892432471546f569c328690251bd6cfc5 100644 (file)
--- a/cmd.h
+++ b/cmd.h
@@ -2,12 +2,10 @@
 #define CMD_H
 
 typedef void (*cgit_cmd_fn)(void);
-typedef void (*cgit_cmd_pre_fn)(void);
 
 struct cgit_cmd {
        const char *name;
        cgit_cmd_fn fn;
-       cgit_cmd_pre_fn pre;
        unsigned int want_repo:1,
                want_vpath:1,
                is_clone:1;