Remove nonsensical size check in mutt_choose_charset()

The charsets parameter is being tokenized by the : delimeter.

The checks against ENCWORD_LEN_MAX and ENCWORD_LEN_MIN make no sense,
and appear to be the result of a large merge a very long time ago
(changeset cb27682966d5).

I can only guess where this check was supposed to be, but it certainly
doesn't belong here.

diff --git a/rfc2047.c b/rfc2047.c
index c566bfcf08947afb634946c013540a630f14c838..8506425450c1ad197b2f87cbd98f67942b0f5630 100644 (file)
--- a/rfc2047.c
+++ b/rfc2047.c
@@ -140,11 +140,7 @@ char *mutt_choose_charset (const char *fromcode, const char *charsets,
     q = strchr (p, ':');
 
     n = q ? q - p : strlen (p);
-
-    if (!n ||
-       /* Assume that we never need more than 12 characters of
-          encoded-text to encode a single character. */
-       n > (ENCWORD_LEN_MAX - ENCWORD_LEN_MIN + 2 - 12))
+    if (!n)
       continue;
 
     t = safe_malloc (n + 1);