diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2014-01-13 11:56:50 (JST) |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2014-01-14 10:00:07 (JST) |
commit | 800380dde797ae35d738a644acdae2fabb9a0d44 (patch) | |
tree | 4770bb2a3eb10021f5d527200ad61a2659a78bea /ui-summary.c | |
parent | f43b228d0bca5791be98ff3fbb2f8743219635b6 (diff) | |
download | cgit-800380dde797ae35d738a644acdae2fabb9a0d44.zip cgit-800380dde797ae35d738a644acdae2fabb9a0d44.tar.gz |
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) | |||
151 | * filesystem, while applying the about-filter. | 151 | * filesystem, while applying the about-filter. |
152 | */ | 152 | */ |
153 | html("<div id='summary'>"); | 153 | html("<div id='summary'>"); |
154 | if (ctx.repo->about_filter) | 154 | cgit_open_filter(ctx.repo->about_filter, filename); |
155 | cgit_open_filter(ctx.repo->about_filter, filename); | ||
156 | |||
157 | if (ref) | 155 | if (ref) |
158 | cgit_print_file(filename, ref, 1); | 156 | cgit_print_file(filename, ref, 1); |
159 | else | 157 | else |
160 | html_include(filename); | 158 | html_include(filename); |
161 | 159 | cgit_close_filter(ctx.repo->about_filter); | |
162 | if (ctx.repo->about_filter) | ||
163 | cgit_close_filter(ctx.repo->about_filter); | ||
164 | 160 | ||
165 | html("</div>"); | 161 | html("</div>"); |
166 | if (free_filename) | 162 | if (free_filename) |