diff options
| author | John Keeping <john@keeping.me.uk> | 2014-01-12 17:13:50 +0000 |
|---|---|---|
| committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2014-01-12 20:20:20 +0100 |
| commit | 3d8a6507ca542881a5e8b30ad6b7068a9c4fdeea (patch) | |
| tree | f95c74d3317546606d72b74db3082769515c5e33 /ui-repolist.c | |
| parent | ui-snapshot: set unused cgit_filter fields to zero (diff) | |
| download | cgit-pink-3d8a6507ca542881a5e8b30ad6b7068a9c4fdeea.tar.gz cgit-pink-3d8a6507ca542881a5e8b30ad6b7068a9c4fdeea.zip | |
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 '')
| -rw-r--r-- | ui-repolist.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/ui-repolist.c b/ui-repolist.c index d4ee279..f622a01 100644 --- a/ui-repolist.c +++ b/ui-repolist.c @@ -331,13 +331,9 @@ void cgit_print_site_readme() { if (!ctx.cfg.root_readme) return; - if (ctx.cfg.about_filter) { - ctx.cfg.about_filter->argv[1] = ctx.cfg.root_readme; - cgit_open_filter(ctx.cfg.about_filter); - } + if (ctx.cfg.about_filter) + cgit_open_filter(ctx.cfg.about_filter, ctx.cfg.root_readme); html_include(ctx.cfg.root_readme); - if (ctx.cfg.about_filter) { + if (ctx.cfg.about_filter) cgit_close_filter(ctx.cfg.about_filter); - ctx.cfg.about_filter->argv[1] = NULL; - } } |