Remove redundant calls to fmt("%s", ...)

After this change there is one remaining call 'fmt("%s", delim)' in
ui-shared.c but is needed as delim is stack allocated and so cannot be
returned from the function.

Signed-off-by: John Keeping <john@keeping.me.uk>

diff --git a/scan-tree.c b/scan-tree.c
index 0d3e0ade7d0be88e7b7b612fbf7ee1ba69f9f023..05caba53af0a88e06e1bbc73c1c34294f8a447db 100644 (file)
--- a/scan-tree.c
+++ b/scan-tree.c
@@ -96,9 +96,9 @@ static void add_repo(const char *base, const char *path, repo_config_fn fn)
                return;
 
        if (base == path)
-               rel = xstrdup(fmt("%s", path));
+               rel = xstrdup(path);
        else
-               rel = xstrdup(fmt("%s", path + strlen(base) + 1));
+               rel = xstrdup(path + strlen(base) + 1);
 
        if (!strcmp(rel + strlen(rel) - 5, "/.git"))
                rel[strlen(rel) - 5] = '\0';

diff --git a/ui-plain.c b/ui-plain.c
index 482d53a89f3d02d231f71fbee5d70874a618c1b4..6b0d84b7a0aec7c8762227edf4c35899d2bd2cce 100644 (file)
--- a/ui-plain.c
+++ b/ui-plain.c
@@ -95,7 +95,7 @@ static int print_object(const unsigned char *sha1, const char *path)
                else
                        ctx.page.mimetype = "text/plain";
        }
-       ctx.page.filename = fmt("%s", path);
+       ctx.page.filename = path;
        ctx.page.size = size;
        ctx.page.etag = sha1_to_hex(sha1);
        cgit_print_http_headers(&ctx);