Changeset 527 for trunk/kernel/kern/process.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/process.c (modified) (1 diff)

trunk/kernel/kern/process.c

@@ -456 +456 @@
 
 /////////////////////////////////////////////////
-char * process_action_str( uint32_t action_type )
-{
-    if     ( action_type == BLOCK_ALL_THREADS   ) return "BLOCK";
-    else if( action_type == UNBLOCK_ALL_THREADS ) return "UNBLOCK";
-    else if( action_type == DELETE_ALL_THREADS  ) return "DELETE";
-    else                                          return "undefined";
+const char * process_action_str( process_sigactions_t action_type )
+{
+  switch ( action_type ) {
+  case BLOCK_ALL_THREADS:   return "BLOCK";
+  case UNBLOCK_ALL_THREADS: return "UNBLOCK";
+  case DELETE_ALL_THREADS:  return "DELETE";
+  default:                  return "undefined";
+  }
 }