Correct CRT_degreeSign buffer size & reduce code size

Make "CRT_degreeSign" a writable buffer that will be updated after the
initDegreeSign() function. This avoids needing a static buffer in the
function.

The appropriate buffer size for "CRT_degreeSign" is (MB_LEN_MAX * 2),
not 4.

One side effect of this change is that "CRT_degreeSign" will lose the
const qualifier in the header (CRT.h).

Signed-off-by: Kang-Che Sung <[email protected]>

Author: Explorer09

CRT.c

@@ -12,6 +12,7 @@ in the source distribution for its full text.
 #include <errno.h>
 #include <fcntl.h>
 #include <langinfo.h>
+#include <limits.h>
 #include <signal.h>
 #include <stdarg.h>
 #include <stdio.h>
@@ -94,21 +95,31 @@ const char* const* CRT_treeStr = CRT_treeStrAscii;
 
 static const Settings* CRT_settings;
 
-const char* CRT_degreeSign;
+#ifdef HAVE_LIBNCURSESW
+# if MB_LEN_MAX >= 3 // Minimum required to support UTF-8 BMP subset
+char CRT_degreeSign[MB_LEN_MAX * 2] = "\xc2\xb0";
+# else
+char CRT_degreeSign[MB_LEN_MAX * 2] = "";
+# endif
+#else
+char CRT_degreeSign[] = "";
+#endif
 
-static const char* initDegreeSign(void) {
+static void initDegreeSign(void) {
 #ifdef HAVE_LIBNCURSESW
+# if MB_LEN_MAX >= 3
    if (CRT_utf8)
-      return "\xc2\xb0";
+      return;
+# endif
 
-   static char buffer[4];
    // this might fail if the current locale does not support wide characters
-   int r = snprintf(buffer, sizeof(buffer), "%lc", 176);
-   if (r > 0)
-      return buffer;
+   int r = snprintf(CRT_degreeSign, sizeof(CRT_degreeSign), "%lc", 176);
+   if (r <= 0)
+      CRT_degreeSign[0] = '\0';
 #endif
 
-   return "";
+   // No-op
+   return;
 }
 
 const int* CRT_colors;
@@ -1145,7 +1156,7 @@ IGNORE_WCASTQUAL_END
 
    CRT_setMouse(settings->enableMouse);
 
-   CRT_degreeSign = initDegreeSign();
+   initDegreeSign();
 }
 
 void CRT_done(void) {

CRT.h

@@ -180,7 +180,7 @@ void CRT_handleSIGSEGV(int signal) ATTR_NORETURN;
 #define KEY_FOCUS_OUT (KEY_MAX + 'O')
 #define KEY_DEL_MAC   127
 
-extern const char* CRT_degreeSign;
+extern char CRT_degreeSign[];
 
 #ifdef HAVE_LIBNCURSESW