commit 5e33015b9b981cb6442a8583753a5a6de91cb2ff
author Maciej Żenczykowski <maze@google.com> Tue Aug 29 13:56:41 2023 +0000
committer Maciej Żenczykowski <maze@google.com> Tue Aug 29 13:57:00 2023 +0000
tree f9f9ef6152a1e5b3982c5535d313b4e10de3edb9
parent 2ef5f2dee6dc4499aa3a7441767e63e2762d835d

FwmarkServer - use a case statement

This is a no-op.

Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Ibba09f903ad84de1a09ff4a918eec60741ba4ab5

server/FwmarkServer.cpp

diff --git a/server/FwmarkServer.cpp b/server/FwmarkServer.cpp
index eb53375..337e4f6 100644
--- a/server/FwmarkServer.cpp
+++ b/server/FwmarkServer.cpp
@@ -87,12 +87,16 @@
 }
 
 static bool hasDestinationAddress(FwmarkCommand::CmdId cmdId, bool redirectSocketCalls) {
-    if (redirectSocketCalls) {
-        return (cmdId == FwmarkCommand::ON_SENDTO || cmdId == FwmarkCommand::ON_CONNECT ||
-                cmdId == FwmarkCommand::ON_SENDMSG || cmdId == FwmarkCommand::ON_SENDMMSG ||
-                cmdId == FwmarkCommand::ON_CONNECT_COMPLETE);
-    } else {
-        return (cmdId == FwmarkCommand::ON_CONNECT_COMPLETE);
+    switch (cmdId) {
+      case FwmarkCommand::ON_SENDTO:
+      case FwmarkCommand::ON_CONNECT:
+      case FwmarkCommand::ON_SENDMSG:
+      case FwmarkCommand::ON_SENDMMSG:
+        return redirectSocketCalls;
+      case FwmarkCommand::ON_CONNECT_COMPLETE:
+        return true;
+      default:
+        return false;
     }
 }