Changeset 527 for trunk/kernel/kern/thread.c

Timestamp:

Aug 30, 2018, 10:26:27 PM (6 years ago)

Author:

viala@…

Message:

Rewrite if-then-else return function into switch case.

For safety reason and performance:

1) Safety: GCC complain with a warning if you forgot an enum variant.
2) code-gen just outperform naive if-then-else.

File:

• trunk/kernel/kern/thread.c (modified) (1 diff)

trunk/kernel/kern/thread.c

@@ -53 +53 @@
 // This function returns a printable string for the thread type.
 //////////////////////////////////////////////////////////////////////////////////////
-char * thread_type_str( uint32_t type )
-{
-    if     ( type == THREAD_USER   ) return "USR";
-    else if( type == THREAD_RPC    ) return "RPC";
-    else if( type == THREAD_DEV    ) return "DEV";
-    else if( type == THREAD_IDLE   ) return "IDL";
-    else                             return "undefined";
+const char * thread_type_str( thread_type_t type )
+{
+  switch ( type ) {
+  case THREAD_USER:   return "USR";
+  case THREAD_RPC:    return "RPC";
+  case THREAD_DEV:    return "DEV";
+  case THREAD_IDLE:   return "IDL";
+  default:            return "undefined";
+  }
 }