[PATCH] Spelling fixes

2017-10-14 Thread Ville Skyttä
Signed-off-by: Ville Skyttä --- cache.c| 2 +- cache.h| 2 +- filters/syntax-highlighting.sh | 2 +- tests/t0109-gitconfig.sh | 10 +- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/cache.c

[PATCH] Use type='search' for the search box

2017-10-14 Thread Ville Skyttä
Signed-off-by: Ville Skyttä --- ui-shared.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ui-shared.c b/ui-shared.c index 07c78a5..9d8f66b 100644 --- a/ui-shared.c +++ b/ui-shared.c @@ -1027,7 +1027,7 @@ void cgit_print_pageheader(void)

[PATCH] filter: pipe_fh should be local

2017-10-14 Thread Ville Skyttä
From: "Jason A. Donenfeld" Signed-off-by: Jason A. Donenfeld --- cgit.h | 1 - filter.c | 13 +++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/cgit.h b/cgit.h index 0b88dcd..005ae63 100644 --- a/cgit.h +++ b/cgit.h @@ -71,7 +71,6

Re: [PATCH 0/4] adding color to ui-blame

2017-10-14 Thread John Keeping
On Sat, Oct 14, 2017 at 04:17:46PM +0200, Jason A. Donenfeld wrote: > This patch set is currently broken, because in the exec filter, > processes like to buffer their output. The result is that the text winds > up at the bottom: > > https://git.zx2c4.com/cgit/blame/cache.c > > If anybody

[PATCH 0/4] adding color to ui-blame

2017-10-14 Thread Jason A. Donenfeld
This patch set is currently broken, because in the exec filter, processes like to buffer their output. The result is that the text winds up at the bottom: https://git.zx2c4.com/cgit/blame/cache.c If anybody has some ideas on how we might enact a flush operation, please pipe up (hah).

[PATCH 2/4] filter: wire up exec_filter to enable function

2017-10-14 Thread Jason A. Donenfeld
We also move pipe_fh to be local, since it's not needed any place else. Signed-off-by: Jason A. Donenfeld --- cgit.h | 1 + filter.c | 27 +-- 2 files changed, 22 insertions(+), 6 deletions(-) diff --git a/cgit.h b/cgit.h index 8da69e7..f9949a7

[PATCH 3/4] filter: wire up lua_filter to enable function

2017-10-14 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld --- filter.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/filter.c b/filter.c index 4deb4de..7b5bef5 100644 --- a/filter.c +++ b/filter.c @@ -353,6 +353,15 @@ static int close_lua_filter(struct cgit_filter *base)

[PATCH 4/4] ui-blame: put source lines through filter

2017-10-14 Thread Jason A. Donenfeld
We toggle the filter on and off so that the control table can avoid going through the filter. Signed-off-by: Jason A. Donenfeld --- ui-blame.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/ui-blame.c b/ui-blame.c index 62cf431..71aec65

Re: [PATCH] parsing: don't clear existing state with empty input

2017-10-14 Thread Jason A. Donenfeld
Looks reasonable to me. Thanks for reporting, Tom, and thanks for the patch John. Applied. ___ CGit mailing list CGit@lists.zx2c4.com https://lists.zx2c4.com/mailman/listinfo/cgit

[PATCH] parsing: don't clear existing state with empty input

2017-10-14 Thread John Keeping
Since commit c699866 (parsing: clear query path before starting, 2017-02-19), we clear the "page" variable simply by calling cgit_parse_url() even if the URL is empty. This breaks a URL like: .../cgit?p=about which is generated when using the "root-readme" configuration option. This