| -rw-r--r-- | cgit.c | 1 | ||||
| -rw-r--r-- | cgit.h | 1 | ||||
| -rw-r--r-- | ui-diff.c | 3 | ||||
| -rw-r--r-- | ui-log.c | 3 | ||||
| -rw-r--r-- | ui-repolist.c | 6 | ||||
| -rw-r--r-- | ui-shared.c | 24 | ||||
| -rw-r--r-- | ui-shared.h | 2 | ||||
| -rw-r--r-- | ui-ssdiff.c | 33 | ||||
| -rw-r--r-- | ui-ssdiff.h | 12 |
9 files changed, 17 insertions, 68 deletions
@@ -298,7 +298,6 @@ static void querystring_cb(const char *name, const char *value) ctx.qry.period = xstrdup(value); } else if (!strcmp(name, "ss")) { ctx.qry.ssdiff = atoi(value); - ctx.qry.has_ssdiff = 1; } else if (!strcmp(name, "all")) { ctx.qry.show_all = atoi(value); } else if (!strcmp(name, "context")) { @@ -137,7 +137,6 @@ struct reflist { struct cgit_query { int has_symref; int has_sha1; - int has_ssdiff; char *raw; char *repo; char *page; @@ -395,7 +395,8 @@ void cgit_print_diff(const char *new_rev, const char *old_rev, } } - use_ssdiff = ctx.qry.has_ssdiff ? ctx.qry.ssdiff : ctx.cfg.ssdiff; + if ((ctx.qry.ssdiff && !ctx.cfg.ssdiff) || (!ctx.qry.ssdiff && ctx.cfg.ssdiff)) + use_ssdiff = 1; if (show_ctrls) cgit_print_diff_ctrls(); @@ -76,8 +76,6 @@ void show_commit_decorations(struct commit *commit) cgit_tag_link(buf, NULL, "tag-deco", ctx.qry.head, buf); } else if (!prefixcmp(deco->name, "refs/remotes/")) { - if (!ctx.repo->enable_remote_branches) - goto next; strncpy(buf, deco->name + 13, sizeof(buf) - 1); cgit_log_link(buf, NULL, "remote-deco", NULL, sha1_to_hex(commit->object.sha1), @@ -90,7 +88,6 @@ void show_commit_decorations(struct commit *commit) sha1_to_hex(commit->object.sha1), ctx.qry.vpath, 0); } -next: deco = deco->next; } } diff --git a/ui-repolist.c b/ui-repolist.c index a09a689..25c36ce 100644 --- a/ui-repolist.c +++ b/ui-repolist.c @@ -118,13 +118,13 @@ void print_header(int columns) } -void print_pager(int items, int pagelen, char *search, char *sort) +void print_pager(int items, int pagelen, char *search) { int i; html("<div class='pager'>"); for(i = 0; i * pagelen < items; i++) cgit_index_link(fmt("[%d]", i+1), fmt("Page %d", i+1), NULL, - search, sort, i * pagelen); + search, i * pagelen); html("</div>"); } @@ -291,7 +291,7 @@ void cgit_print_repolist() if (!hits) cgit_print_error("No repositories found"); else if (hits > ctx.cfg.max_repo_count) - print_pager(hits, ctx.cfg.max_repo_count, ctx.qry.search, ctx.qry.sort); + print_pager(hits, ctx.cfg.max_repo_count, ctx.qry.search); cgit_print_docend(); } diff --git a/ui-shared.c b/ui-shared.c index 85c2096..5aa9119 100644 --- a/ui-shared.c +++ b/ui-shared.c @@ -133,7 +133,7 @@ char *cgit_currurl() return fmt("%s/", ctx.cfg.virtual_root); } -static void site_url(const char *page, const char *search, const char *sort, int ofs) +static void site_url(const char *page, const char *search, int ofs) { char *delim = "?"; @@ -154,12 +154,6 @@ static void site_url(const char *page, const char *search, const char *sort, int html_attr(search); delim = "&"; } - if (sort) { - html(delim); - html("s="); - html_attr(sort); - delim = "&"; - } if (ofs) { html(delim); htmlf("ofs=%d", ofs); @@ -167,7 +161,7 @@ static void site_url(const char *page, const char *search, const char *sort, int } static void site_link(const char *page, const char *name, const char *title, - const char *class, const char *search, const char *sort, int ofs) + const char *class, const char *search, int ofs) { html("<a"); if (title) { @@ -181,16 +175,16 @@ static void site_link(const char *page, const char *name, const char *title, html("'"); } html(" href='"); - site_url(page, search, sort, ofs); + site_url(page, search, ofs); html("'>"); html_txt(name); html("</a>"); } void cgit_index_link(const char *name, const char *title, const char *class, - const char *pattern, const char *sort, int ofs) + const char *pattern, int ofs) { - site_link(NULL, name, title, class, pattern, sort, ofs); + site_link(NULL, name, title, class, pattern, ofs); } static char *repolink(const char *title, const char *class, const char *page, @@ -434,7 +428,7 @@ void cgit_self_link(char *name, const char *title, const char *class, struct cgit_context *ctx) { if (!strcmp(ctx->qry.page, "repolist")) - return cgit_index_link(name, title, class, ctx->qry.search, ctx->qry.sort, + return cgit_index_link(name, title, class, ctx->qry.search, ctx->qry.ofs); else if (!strcmp(ctx->qry.page, "summary")) return cgit_summary_link(name, title, class, ctx->qry.head); @@ -788,7 +782,7 @@ static void print_header(struct cgit_context *ctx) html("<td class='main'>"); if (ctx->repo) { - cgit_index_link("index", NULL, NULL, NULL, NULL, 0); + cgit_index_link("index", NULL, NULL, NULL, 0); html(" : "); cgit_summary_link(ctx->repo->name, ctx->repo->name, NULL, NULL); html("</td><td class='form'>"); @@ -864,10 +858,10 @@ void cgit_print_pageheader(struct cgit_context *ctx) html("<input type='submit' value='search'/>\n"); html("</form>\n"); } else { - site_link(NULL, "index", NULL, hc(ctx, "repolist"), NULL, NULL, 0); + site_link(NULL, "index", NULL, hc(ctx, "repolist"), NULL, 0); if (ctx->cfg.root_readme) site_link("about", "about", NULL, hc(ctx, "about"), - NULL, NULL, 0); + NULL, 0); html("</td><td class='form'>"); html("<form method='get' action='"); html_attr(cgit_rooturl()); diff --git a/ui-shared.h b/ui-shared.h index 865bd45..3cc1258 100644 --- a/ui-shared.h +++ b/ui-shared.h @@ -11,7 +11,7 @@ extern char *cgit_pageurl(const char *reponame, const char *pagename, const char *query); extern void cgit_index_link(const char *name, const char *title, - const char *class, const char *pattern, const char *sort, int ofs); + const char *class, const char *pattern, int ofs); extern void cgit_summary_link(const char *name, const char *title, const char *class, const char *head); extern void cgit_tag_link(const char *name, const char *title, diff --git a/ui-ssdiff.c b/ui-ssdiff.c index 9fb5b11..2481585 100644 --- a/ui-ssdiff.c +++ b/ui-ssdiff.c @@ -2,12 +2,10 @@ #include "html.h" #include "ui-shared.h" #include "ui-diff.h" -#include "ui-ssdiff.h" extern int use_ssdiff; static int current_old_line, current_new_line; -static int **L = NULL; struct deferred_lines { int line_no; @@ -18,42 +16,16 @@ struct deferred_lines { static struct deferred_lines *deferred_old, *deferred_old_last; static struct deferred_lines *deferred_new, *deferred_new_last; -static void create_or_reset_lcs_table() -{ - int i; - - if (L != NULL) { - memset(*L, 0, sizeof(*L) * MAX_SSDIFF_SIZE); - return; - } - - // xcalloc will die if we ran out of memory; - // not very helpful for debugging - L = (int**)xcalloc(MAX_SSDIFF_M, sizeof(int *)); - *L = (int*)xcalloc(MAX_SSDIFF_SIZE, sizeof(int)); - - for (i = 1; i < MAX_SSDIFF_M; i++) { - L[i] = *L + i * MAX_SSDIFF_N; - } -} - static char *longest_common_subsequence(char *A, char *B) { int i, j, ri; int m = strlen(A); int n = strlen(B); - int tmp1, tmp2, length; + int L[m + 1][n + 1]; + int tmp1, tmp2; int lcs_length; char *result; - length = (m + 1) * (n + 1); - - // We bail if the lines are too long - if (length > MAX_SSDIFF_SIZE) - return NULL; - - create_or_reset_lcs_table(); - for (i = m; i >= 0; i--) { for (j = n; j >= 0; j--) { if (A[i] == '\0' || B[j] == '\0') { @@ -87,7 +59,6 @@ static char *longest_common_subsequence(char *A, char *B) j += 1; } } - return result; } diff --git a/ui-ssdiff.h b/ui-ssdiff.h index 88627e2..64b4b12 100644 --- a/ui-ssdiff.h +++ b/ui-ssdiff.h @@ -1,18 +1,6 @@ #ifndef UI_SSDIFF_H #define UI_SSDIFF_H -/* - * ssdiff line limits - */ -#ifndef MAX_SSDIFF_M -#define MAX_SSDIFF_M 128 -#endif - -#ifndef MAX_SSDIFF_N -#define MAX_SSDIFF_N 128 -#endif -#define MAX_SSDIFF_SIZE ((MAX_SSDIFF_M) * (MAX_SSDIFF_N)) - extern void cgit_ssdiff_print_deferred_lines(); extern void cgit_ssdiff_line_cb(char *line, int len); |