From 3af246bb3291d8568f6d110884fa55ee5fd20221 Mon Sep 17 00:00:00 2001 From: David Robillard Date: Thu, 18 Aug 2022 01:22:18 -0400 Subject: Avoid "else" after "return", "break", and "continue" --- src/SocketReader.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src/SocketReader.cpp') diff --git a/src/SocketReader.cpp b/src/SocketReader.cpp index d4b400bb..c7b64166 100644 --- a/src/SocketReader.cpp +++ b/src/SocketReader.cpp @@ -167,7 +167,9 @@ SocketReader::run() if (ret == -1 || (pfd.revents & (POLLERR|POLLHUP|POLLNVAL))) { on_hangup(); break; // Hangup - } else if (!ret) { + } + + if (!ret) { continue; // No data, shouldn't happen } @@ -178,7 +180,9 @@ SocketReader::run() SerdStatus st = serd_reader_read_chunk(reader); if (st == SERD_FAILURE || !_msg_node) { continue; // Read nothing, e.g. just whitespace - } else if (st) { + } + + if (st) { _world.log().error("Read error: %1%\n", serd_strerror(st)); continue; } -- cgit v1.2.1