commit 3e66184f23d855b361cf1661a58bee26580c4ba3
author guoshichao <guoshichao@xiaomi.com> Mon Sep 09 14:57:54 2024 +0800
committer Xiang Xiao <xiaoxiang781216@gmail.com> Fri Sep 27 00:20:53 2024 +0800
tree 2a689528968d2434702c839d2896ae7df99cdd2a
parent c9223469d410961ad7a401406d8f36d3277a526b

pthread_mutexattr_setprotocol: remove the unreached dead code

Signed-off-by: guoshichao <guoshichao@xiaomi.com>

libs/libc/pthread/pthread_mutexattr_setprotocol.c

diff --git a/libs/libc/pthread/pthread_mutexattr_setprotocol.c b/libs/libc/pthread/pthread_mutexattr_setprotocol.c
index 28c5377..9233891 100644
--- a/libs/libc/pthread/pthread_mutexattr_setprotocol.c
+++ b/libs/libc/pthread/pthread_mutexattr_setprotocol.c
@@ -54,38 +54,33 @@
   linfo("attr=%p protocol=%d\n", attr, protocol);
   DEBUGASSERT(attr != NULL);
 
-  if (protocol >= PTHREAD_PRIO_NONE && protocol <= PTHREAD_PRIO_PROTECT)
+  switch (protocol)
     {
-      switch (protocol)
-        {
-          case PTHREAD_PRIO_NONE:
+      case PTHREAD_PRIO_NONE:
 #if defined(CONFIG_PRIORITY_INHERITANCE) || defined(CONFIG_PRIORITY_PROTECT)
-            attr->proto = PTHREAD_PRIO_INHERIT;
+        attr->proto = PTHREAD_PRIO_INHERIT;
 #endif
-            break;
+        break;
 
-          case PTHREAD_PRIO_INHERIT:
+      case PTHREAD_PRIO_INHERIT:
 #ifdef CONFIG_PRIORITY_INHERITANCE
-            attr->proto = PTHREAD_PRIO_INHERIT;
-            break;
+        attr->proto = PTHREAD_PRIO_INHERIT;
+        break;
 #else
-            return ENOTSUP;
+        return ENOTSUP;
 #endif /* CONFIG_PRIORITY_INHERITANCE */
 
-          case PTHREAD_PRIO_PROTECT:
+      case PTHREAD_PRIO_PROTECT:
 #ifdef CONFIG_PRIORITY_PROTECT
-            attr->proto = PTHREAD_PRIO_PROTECT;
-            break;
+        attr->proto = PTHREAD_PRIO_PROTECT;
+        break;
 #else
-            return ENOTSUP;
+        return ENOTSUP;
 #endif /* CONFIG_PRIORITY_PROTECT */
 
-          default:
-            return ENOTSUP;
-        }
-
-      return OK;
+      default:
+        return EINVAL;
     }
 
-  return EINVAL;
+  return OK;
 }