diff options
| author | Jason A. Donenfeld | 2014-01-13 03:56:50 +0100 |
|---|---|---|
| committer | Jason A. Donenfeld | 2014-01-14 02:00:07 +0100 |
| commit | 800380dde797ae35d738a644acdae2fabb9a0d44 (patch) | |
| tree | 4770bb2a3eb10021f5d527200ad61a2659a78bea /filter.c | |
| parent | f43b228d0bca5791be98ff3fbb2f8743219635b6 (diff) | |
| download | cgit-800380dde797ae35d738a644acdae2fabb9a0d44.tar.gz cgit-800380dde797ae35d738a644acdae2fabb9a0d44.tar.bz2 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 'filter.c')
| -rw-r--r-- | filter.c | 4 |
1 files changed, 4 insertions, 0 deletions
| @@ -351,6 +351,8 @@ int cgit_open_filter(struct cgit_filter *filter, ...) | |||
| 351 | { | 351 | { |
| 352 | int result; | 352 | int result; |
| 353 | va_list ap; | 353 | va_list ap; |
| 354 | if (!filter) | ||
| 355 | return 0; | ||
| 354 | va_start(ap, filter); | 356 | va_start(ap, filter); |
| 355 | result = filter->open(filter, ap); | 357 | result = filter->open(filter, ap); |
| 356 | va_end(ap); | 358 | va_end(ap); |
| @@ -359,6 +361,8 @@ int cgit_open_filter(struct cgit_filter *filter, ...) | |||
| 359 | 361 | ||
| 360 | int cgit_close_filter(struct cgit_filter *filter) | 362 | int cgit_close_filter(struct cgit_filter *filter) |
| 361 | { | 363 | { |
| 364 | if (!filter) | ||
| 365 | return 0; | ||
| 362 | return filter->close(filter); | 366 | return filter->close(filter); |
| 363 | } | 367 | } |
| 364 | 368 | ||
