diff options
| author | Jason A. Donenfeld <Jason@zx2c4.com> | 2014-01-13 02:56:50 +0000 |
|---|---|---|
| committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2014-01-14 01:00:07 +0000 |
| commit | 800380dde797ae35d738a644acdae2fabb9a0d44 (patch) | |
| tree | 4770bb2a3eb10021f5d527200ad61a2659a78bea /ui-summary.c | |
| parent | f43b228d0bca5791be98ff3fbb2f8743219635b6 (diff) | |
| download | cgit-800380dde797ae35d738a644acdae2fabb9a0d44.tar cgit-800380dde797ae35d738a644acdae2fabb9a0d44.tar.gz cgit-800380dde797ae35d738a644acdae2fabb9a0d44.zip | |
filter: return on null filter from open and close
So that we don't have to include the if(filter) open_filter(filter)
block everywhere, we introduce the guard in the function itself. This
should simplify quite a bit of code.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Diffstat (limited to 'ui-summary.c')
| -rw-r--r-- | ui-summary.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/ui-summary.c b/ui-summary.c index 725f3ab..ddd8f1b 100644 --- a/ui-summary.c +++ b/ui-summary.c @@ -151,16 +151,12 @@ void cgit_print_repo_readme(char *path) * filesystem, while applying the about-filter. */ html("<div id='summary'>"); - if (ctx.repo->about_filter) - cgit_open_filter(ctx.repo->about_filter, filename); - + cgit_open_filter(ctx.repo->about_filter, filename); if (ref) cgit_print_file(filename, ref, 1); else html_include(filename); - - if (ctx.repo->about_filter) - cgit_close_filter(ctx.repo->about_filter); + cgit_close_filter(ctx.repo->about_filter); html("</div>"); if (free_filename) |