Merge branch 'jh/context-lines'

Conflicts:
	cgit.c
	cgit.h
lh/pretty-blob-view
Lars Hjemli 14 years ago
commit 6f92f332e6

@ -258,6 +258,8 @@ static void querystring_cb(const char *name, const char *value)
ctx.qry.ssdiff = atoi(value); ctx.qry.ssdiff = atoi(value);
} else if (!strcmp(name, "all")) { } else if (!strcmp(name, "all")) {
ctx.qry.show_all = atoi(value); ctx.qry.show_all = atoi(value);
} else if (!strcmp(name, "context")) {
ctx.qry.context = atoi(value);
} }
} }

@ -147,6 +147,7 @@ struct cgit_query {
int showmsg; int showmsg;
int ssdiff; int ssdiff;
int show_all; int show_all;
int context;
char *vpath; char *vpath;
}; };
@ -278,7 +279,7 @@ extern void *cgit_free_commitinfo(struct commitinfo *info);
extern int cgit_diff_files(const unsigned char *old_sha1, extern int cgit_diff_files(const unsigned char *old_sha1,
const unsigned char *new_sha1, const unsigned char *new_sha1,
unsigned long *old_size, unsigned long *new_size, unsigned long *old_size, unsigned long *new_size,
int *binary, linediff_fn fn); int *binary, int context, linediff_fn fn);
extern void cgit_diff_tree(const unsigned char *old_sha1, extern void cgit_diff_tree(const unsigned char *old_sha1,
const unsigned char *new_sha1, const unsigned char *new_sha1,

@ -263,7 +263,8 @@ int filediff_cb(void *priv, mmbuffer_t *mb, int nbuf)
int cgit_diff_files(const unsigned char *old_sha1, int cgit_diff_files(const unsigned char *old_sha1,
const unsigned char *new_sha1, unsigned long *old_size, const unsigned char *new_sha1, unsigned long *old_size,
unsigned long *new_size, int *binary, linediff_fn fn) unsigned long *new_size, int *binary, int context,
linediff_fn fn)
{ {
mmfile_t file1, file2; mmfile_t file1, file2;
xpparam_t diff_params; xpparam_t diff_params;
@ -290,7 +291,7 @@ int cgit_diff_files(const unsigned char *old_sha1,
memset(&emit_params, 0, sizeof(emit_params)); memset(&emit_params, 0, sizeof(emit_params));
memset(&emit_cb, 0, sizeof(emit_cb)); memset(&emit_cb, 0, sizeof(emit_cb));
diff_params.flags = XDF_NEED_MINIMAL; diff_params.flags = XDF_NEED_MINIMAL;
emit_params.ctxlen = 3; emit_params.ctxlen = context > 0 ? context : 3;
emit_params.flags = XDL_EMIT_FUNCNAMES; emit_params.flags = XDL_EMIT_FUNCNAMES;
emit_cb.outf = filediff_cb; emit_cb.outf = filediff_cb;
emit_cb.priv = fn; emit_cb.priv = fn;

@ -127,7 +127,7 @@ static void inspect_filepair(struct diff_filepair *pair)
lines_added = 0; lines_added = 0;
lines_removed = 0; lines_removed = 0;
cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, &new_size, cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, &new_size,
&binary, count_diff_lines); &binary, 0, count_diff_lines);
if (files >= slots) { if (files >= slots) {
if (slots == 0) if (slots == 0)
slots = 4; slots = 4;
@ -156,13 +156,21 @@ static void inspect_filepair(struct diff_filepair *pair)
void cgit_print_diffstat(const unsigned char *old_sha1, void cgit_print_diffstat(const unsigned char *old_sha1,
const unsigned char *new_sha1, const char *prefix) const unsigned char *new_sha1, const char *prefix)
{ {
int i; int i, save_context = ctx.qry.context;
html("<div class='diffstat-header'>"); html("<div class='diffstat-header'>");
cgit_diff_link("Diffstat", NULL, NULL, ctx.qry.head, ctx.qry.sha1, cgit_diff_link("Diffstat", NULL, NULL, ctx.qry.head, ctx.qry.sha1,
ctx.qry.sha2, NULL, 0); ctx.qry.sha2, NULL, 0);
if (prefix) if (prefix)
htmlf(" (limited to '%s')", prefix); htmlf(" (limited to '%s')", prefix);
html(" (");
ctx.qry.context = (save_context > 0 ? save_context : 3) << 1;
cgit_self_link("more", NULL, NULL, &ctx);
html("/");
ctx.qry.context = (save_context > 3 ? save_context : 3) >> 1;
cgit_self_link("less", NULL, NULL, &ctx);
ctx.qry.context = save_context;
html(" context)");
html("</div>"); html("</div>");
html("<table summary='diffstat' class='diffstat'>"); html("<table summary='diffstat' class='diffstat'>");
max_changes = 0; max_changes = 0;
@ -288,7 +296,7 @@ static void filepair_cb(struct diff_filepair *pair)
return; return;
} }
if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size,
&new_size, &binary, print_line_fn)) &new_size, &binary, ctx.qry.context, print_line_fn))
cgit_print_error("Error running diff"); cgit_print_error("Error running diff");
if (binary) { if (binary) {
if (use_ssdiff) if (use_ssdiff)

@ -33,7 +33,7 @@ void inspect_files(struct diff_filepair *pair)
files++; files++;
if (ctx.repo->enable_log_linecount) if (ctx.repo->enable_log_linecount)
cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size,
&new_size, &binary, count_lines); &new_size, &binary, 0, count_lines);
} }
void show_commit_decorations(struct commit *commit) void show_commit_decorations(struct commit *commit)

@ -71,7 +71,7 @@ static void filepair_cb(struct diff_filepair *pair)
return; return;
} }
if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size,
&new_size, &binary, print_line)) &new_size, &binary, 0, print_line))
html("Error running diff"); html("Error running diff");
if (binary) if (binary)
html("Binary files differ\n"); html("Binary files differ\n");

@ -341,6 +341,13 @@ void cgit_commit_link(char *name, const char *title, const char *class,
if ((ctx.qry.ssdiff && !toggle_ssdiff) || (!ctx.qry.ssdiff && toggle_ssdiff)) { if ((ctx.qry.ssdiff && !toggle_ssdiff) || (!ctx.qry.ssdiff && toggle_ssdiff)) {
html(delim); html(delim);
html("ss=1"); html("ss=1");
delim = "&amp;";
}
if (ctx.qry.context > 0 && ctx.qry.context != 3) {
html(delim);
html("context=");
htmlf("%d", ctx.qry.context);
delim = "&amp;";
} }
html("'>"); html("'>");
html_txt(name); html_txt(name);
@ -382,6 +389,13 @@ void cgit_diff_link(const char *name, const char *title, const char *class,
if ((ctx.qry.ssdiff && !toggle_ssdiff) || (!ctx.qry.ssdiff && toggle_ssdiff)) { if ((ctx.qry.ssdiff && !toggle_ssdiff) || (!ctx.qry.ssdiff && toggle_ssdiff)) {
html(delim); html(delim);
html("ss=1"); html("ss=1");
delim = "&amp;";
}
if (ctx.qry.context > 0 && ctx.qry.context != 3) {
html(delim);
html("context=");
htmlf("%d", ctx.qry.context);
delim = "&amp;";
} }
html("'>"); html("'>");
html_txt(name); html_txt(name);

Loading…
Cancel
Save