diff options
author | John Keeping <john@keeping.me.uk> | 2014-01-13 02:13:50 (JST) |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2014-01-13 04:20:20 (JST) |
commit | 3d8a6507ca542881a5e8b30ad6b7068a9c4fdeea (patch) | |
tree | f95c74d3317546606d72b74db3082769515c5e33 /ui-summary.c | |
parent | da218fcd9eb554a8405ca72e84bcc0feb371194f (diff) | |
download | cgit-3d8a6507ca542881a5e8b30ad6b7068a9c4fdeea.zip cgit-3d8a6507ca542881a5e8b30ad6b7068a9c4fdeea.tar.gz |
filter: pass extra arguments via cgit_open_filter
This avoids poking into the filter data structure at various points in
the code. We rely on the fact that the number of arguments is fixed
based on the filter type (set in cgit_new_filter) and that the call
sites all know which filter type they're using.
Signed-off-by: John Keeping <john@keeping.me.uk>
Diffstat (limited to 'ui-summary.c')
-rw-r--r-- | ui-summary.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/ui-summary.c b/ui-summary.c index 63a5a75..725f3ab 100644 --- a/ui-summary.c +++ b/ui-summary.c | |||
@@ -151,18 +151,17 @@ 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 | if (ctx.repo->about_filter) |
155 | ctx.repo->about_filter->argv[1] = filename; | 155 | cgit_open_filter(ctx.repo->about_filter, filename); |
156 | cgit_open_filter(ctx.repo->about_filter); | 156 | |
157 | } | ||
158 | if (ref) | 157 | if (ref) |
159 | cgit_print_file(filename, ref, 1); | 158 | cgit_print_file(filename, ref, 1); |
160 | else | 159 | else |
161 | html_include(filename); | 160 | html_include(filename); |
162 | if (ctx.repo->about_filter) { | 161 | |
162 | if (ctx.repo->about_filter) | ||
163 | cgit_close_filter(ctx.repo->about_filter); | 163 | cgit_close_filter(ctx.repo->about_filter); |
164 | ctx.repo->about_filter->argv[1] = NULL; | 164 | |
165 | } | ||
166 | html("</div>"); | 165 | html("</div>"); |
167 | if (free_filename) | 166 | if (free_filename) |
168 | free(filename); | 167 | free(filename); |