rgw: eliminate one unnecessary case statement

0x21 '!' is the first character that doesn't need encoding, so we can
expand the lower bound check.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

diff --git a/src/rgw/rgw_common.cc b/src/rgw/rgw_common.cc
index e51b08889cb96278bce4eba09837aafcd861cc43..cb87c18b93d71898c55cd52cab51969dbeb0ffd4 100644 (file)
--- a/src/rgw/rgw_common.cc
+++ b/src/rgw/rgw_common.cc
@@ -714,11 +714,10 @@ static void escape_char(char c, string& dst)
 
 static bool char_needs_url_encoding(char c)
 {
-  if (c < 0x20 || c >= 0x7f)
+  if (c <= 0x20 || c >= 0x7f)
     return true;
 
   switch (c) {
-    case 0x20:
     case 0x22:
     case 0x23:
     case 0x25: