[Bf-blender-cvs] [5496d8cd361] blender-v2.92-release: Windows: Fix wchar_t truncation
Ray Molenkamp
noreply at git.blender.org
Tue Jan 26 22:56:47 CET 2021
Commit: 5496d8cd361385268316f91afa150e69b5345ab0
Author: Ray Molenkamp
Date: Tue Jan 26 14:56:39 2021 -0700
Branches: blender-v2.92-release
https://developer.blender.org/rB5496d8cd361385268316f91afa150e69b5345ab0
Windows: Fix wchar_t truncation
BLI_strncpy_wchar_from_utf8 made the assumption that
wchar_t is UTF-32 bit regardless of environment, while
this holds true on both mac and linux, on windows
wchar_t is actually actually UTF-16.
This resulted in the upper 16 bits being dropped from
from some string conversions and prevented blender
from starting when installed in a path with unicode
code-points over 0xffff.
There was also a fair bit of code duplication between
BLI_strncpy_wchar_from_utf8 and BLI_str_utf8_as_unicode_and_size
this change essentially removes all logic from
BLI_strncpy_wchar_from_utf8 and calls the right function
for the right environment.
Reviewed By: brecht , Robert Guetzkow
Differential Revision: https://developer.blender.org/D9822
===================================================================
M source/blender/blenlib/intern/string_utf8.c
===================================================================
diff --git a/source/blender/blenlib/intern/string_utf8.c b/source/blender/blenlib/intern/string_utf8.c
index 0a723a623f0..a637a5f24e6 100644
--- a/source/blender/blenlib/intern/string_utf8.c
+++ b/source/blender/blenlib/intern/string_utf8.c
@@ -33,7 +33,9 @@
#include "BLI_utildefines.h"
#include "BLI_string_utf8.h" /* own include */
-
+#ifdef WIN32
+# include "utfconv.h"
+#endif
#ifdef __GNUC__
# pragma GCC diagnostic error "-Wsign-conversion"
#endif
@@ -393,38 +395,11 @@ size_t BLI_strncpy_wchar_from_utf8(wchar_t *__restrict dst_w,
const char *__restrict src_c,
const size_t maxncpy)
{
- const size_t maxlen = maxncpy - 1;
- size_t len = 0;
-
- BLI_assert(maxncpy != 0);
-
-#ifdef DEBUG_STRSIZE
- memset(dst_w, 0xff, sizeof(*dst_w) * maxncpy);
+#ifdef WIN32
+ return conv_utf_8_to_16(src_c, dst_w, maxncpy);
+#else
+ return BLI_str_utf8_as_utf32((char32_t *)dst_w, src_c, maxncpy);
#endif
-
- while (*src_c && len != maxlen) {
- size_t step = 0;
- uint unicode = BLI_str_utf8_as_unicode_and_size(src_c, &step);
- if (unicode != BLI_UTF8_ERR) {
- /* TODO: `wchar_t` type is an implementation-defined and may represent
- * 16-bit or 32-bit depending on operating system.
- * So the ideal would be to do the corresponding encoding.
- * But for now just assert that it has no conflicting use. */
- BLI_assert(step <= sizeof(wchar_t));
- *dst_w = (wchar_t)unicode;
- src_c += step;
- }
- else {
- *dst_w = '?';
- src_c = BLI_str_find_next_char_utf8(src_c, NULL);
- }
- dst_w++;
- len++;
- }
-
- *dst_w = 0;
-
- return len;
}
/* end wchar_t / utf8 functions */
More information about the Bf-blender-cvs
mailing list