Merge pull request #1598 from Explorer09/init-degree-sign

CRT_degreeSign code "shrink"

Author: BenBE

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 || (size_t)r >= sizeof(CRT_degreeSign))
+      CRT_degreeSign[0] = '\0';
 #endif
 
-   return "";
+   // No-op
+   return;
 }
 
 const int* CRT_colors;
@@ -1265,7 +1276,7 @@ IGNORE_WCASTQUAL_END
 
    CRT_setMouse(settings->enableMouse);
 
-   CRT_degreeSign = initDegreeSign();
+   initDegreeSign();
 }
 
 void CRT_done(void) {

CRT.h

@@ -182,7 +182,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