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

trunk/kernel/devices/dev_txt.c

@@ -49 +49 @@
 
 ////////////////////////////////////////
-char * dev_txt_type_str( uint32_t type )
-{
-    if     ( type == TXT_SYNC_WRITE ) return "TXT_SYNC_WRITE";
-    else if( type == TXT_READ       ) return "TXT_READ";
-    else if( type == TXT_WRITE      ) return "TXT_WRITE";
-    else                              return "undefined"; 
+const char * dev_txt_type_str( dev_txt_cmd_t type )
+{
+  switch (type) {
+    case (TXT_SYNC_WRITE): return "TXT_SYNC_WRITE";
+    case (TXT_READ):       return "TXT_READ";
+    case (TXT_WRITE):      return "TXT_WRITE";
+    default:               return "undefined";
+  }
 }