Convert an if/else chain to a switch. Add default state.

The default was the bigger issue.  Defaulting to NOT_STORED is
probably OK, but if a new storage type comes up, I'd rather see failures.

1 files changed, 13 insertions, 4 deletions

diff --git a/memcached.c b/memcached.c
index 8ec2c4a..8381cc8 100644
--- a/memcached.c
+++ b/memcached.c
@@ -860,14 +860,23 @@ static void complete_nread_ascii(conn *c) {
       }
 #endif
 
-      if (ret == STORED)
+      switch (ret) {
+      case STORED:
           out_string(c, "STORED");
-      else if(ret == EXISTS)
+          break;
+      case EXISTS:
           out_string(c, "EXISTS");
-      else if(ret == NOT_FOUND)
+          break;
+      case NOT_FOUND:
           out_string(c, "NOT_FOUND");
-      else
+          break;
+      case NOT_STORED:
           out_string(c, "NOT_STORED");
+          break;
+      default:
+          out_string(c, "SERVER_ERROR Unhandled storage type.");
+      }
+
     }
 
     item_remove(c->item);       /* release the c->item reference */