aboutsummaryrefslogtreecommitdiffstats
path: root/builtin.c
diff options
context:
space:
mode:
authorAndrew J. Schorr <aschorr@telemetry-investments.com>2021-12-02 09:14:22 -0500
committerAndrew J. Schorr <aschorr@telemetry-investments.com>2021-12-02 09:14:22 -0500
commit7d8a75a766b17fe395c0a82050d314c2dea57f3e (patch)
tree73bbb9f6255684bf69e12b80af622fc7a4a965a4 /builtin.c
parentf77e1318c515d495ac9c08bdfdf2dadf79a9649f (diff)
downloadegawk-7d8a75a766b17fe395c0a82050d314c2dea57f3e.tar.gz
egawk-7d8a75a766b17fe395c0a82050d314c2dea57f3e.tar.bz2
egawk-7d8a75a766b17fe395c0a82050d314c2dea57f3e.zip
Don't use return in a call to a void function.
Diffstat (limited to 'builtin.c')
-rw-r--r--builtin.c12
1 files changed, 8 insertions, 4 deletions
diff --git a/builtin.c b/builtin.c
index d77d276e..91cd142f 100644
--- a/builtin.c
+++ b/builtin.c
@@ -151,10 +151,14 @@ efwrite(const void *ptr,
{
errno = 0;
if (rp != NULL) {
- if (rp->output.gawk_fwrite(ptr, size, count, fp, rp->output.opaque) != count)
- return wrerror(fp, from, rp);
- } else if (fwrite(ptr, size, count, fp) != count)
- return wrerror(fp, from, rp);
+ if (rp->output.gawk_fwrite(ptr, size, count, fp, rp->output.opaque) != count) {
+ wrerror(fp, from, rp);
+ return;
+ }
+ } else if (fwrite(ptr, size, count, fp) != count) {
+ wrerror(fp, from, rp);
+ return;
+ }
if (flush
&& ((fp == stdout && output_is_tty)
|| (rp != NULL && (rp->flag & RED_NOBUF) != 0)))