From ee5324aa04bb46a9a8e1ad23307efa85e05879b6 Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Wed, 14 Mar 2018 17:48:29 +0900 Subject: tree-wide: voidify pager_open() Even if pager_open() fails, in general, we should continue the operations. All erroneous cases in pager_open() show log message in the function. So, it is not necessary to check the returned value. --- src/cgls/cgls.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'src/cgls') diff --git a/src/cgls/cgls.c b/src/cgls/cgls.c index bd8c6a0059..1e92dabe0c 100644 --- a/src/cgls/cgls.c +++ b/src/cgls/cgls.c @@ -176,11 +176,9 @@ int main(int argc, char *argv[]) { if (r <= 0) goto finish; - if (!arg_no_pager) { - r = pager_open(arg_no_pager, false); - if (r > 0 && arg_full < 0) - arg_full = true; - } + r = pager_open(arg_no_pager, false); + if (r > 0 && arg_full < 0) + arg_full = true; output_flags = arg_all * OUTPUT_SHOW_ALL | -- cgit v1.2.3