diff options
| author | 2014-01-13 11:56:50 (JST) | |
|---|---|---|
| committer | 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) |
