From: Ralph Corderoy <ralph@inputplus.co.uk>
Date: Thu, 20 Oct 2016 22:21:45 +0000 (+0100)
Subject: Don't `else' after return.  Simplify control flow.
X-Git-Url: https://diplodocus.org/git/nmh/commitdiff_plain/db744b85cee9eccd9840823d06fa8ea1ff124993?hp=cec6ee584ab027c7c536fb0bb62e768b40a4e94c

Don't `else' after return.  Simplify control flow.
---

diff --git a/sbr/mime_type.c b/sbr/mime_type.c
index 23ccf038..e4f3e5b8 100644
--- a/sbr/mime_type.c
+++ b/sbr/mime_type.c
@@ -124,9 +124,8 @@ get_file_info(const char *proc, const char *file_name) {
             advise(NULL, "filenames containing both single and double quotes "
                    "are unsupported for attachment");
             return NULL;
-        } else {
-            quotec = "\"";
         }
+        quotec = "\"";
     }
 
     if ((cmd = concat(proc, " ", quotec, file_name, quotec, NULL))) {