aboutsummaryrefslogtreecommitdiffstats
path: root/filter.c
diff options
context:
space:
mode:
authorGravatar Jason A. Donenfeld <Jason@zx2c4.com>2014-01-13 11:56:50 (JST)
committerGravatar Jason A. Donenfeld <Jason@zx2c4.com>2014-01-14 10:00:07 (JST)
commit800380dde797ae35d738a644acdae2fabb9a0d44 (patch)
tree4770bb2a3eb10021f5d527200ad61a2659a78bea /filter.c
parentf43b228d0bca5791be98ff3fbb2f8743219635b6 (diff)
downloadcgit-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 'filter.c')
-rw-r--r--filter.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/filter.c b/filter.c
index 3702585..7983737 100644
--- a/filter.c
+++ b/filter.c
@@ -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
360int cgit_close_filter(struct cgit_filter *filter) 362int 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