Re: [U-Boot] [PATCH 3/5] vsprintf.c: add UTF-16 string (%ls) support

[Simon Glass]

On 9 August 2017 at 17:14, Rob Clark  wrote:
> This is convenient for efi_loader which deals a lot with UTF-16.  Only
> enabled with CC_SHORT_WCHAR, leaving room to add a UTF-32 version when
> CC_SHORT_WCHAR is not enabled.
>
> Signed-off-by: Rob Clark 
> ---
>  examples/api/Makefile |  1 +
>  lib/vsprintf.c| 31 +--
>  2 files changed, 30 insertions(+), 2 deletions(-)
>

Reviewed-by: Simon Glass 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 3/5] vsprintf.c: add UTF-16 string (%ls) support

[Rob Clark]

This is convenient for efi_loader which deals a lot with UTF-16.  Only
enabled with CC_SHORT_WCHAR, leaving room to add a UTF-32 version when
CC_SHORT_WCHAR is not enabled.

Signed-off-by: Rob Clark 
---
 examples/api/Makefile |  1 +
 lib/vsprintf.c| 31 +--
 2 files changed, 30 insertions(+), 2 deletions(-)

diff --git a/examples/api/Makefile b/examples/api/Makefile
index dab6398bab..87c15d0f68 100644
--- a/examples/api/Makefile
+++ b/examples/api/Makefile
@@ -34,6 +34,7 @@ EXT_COBJ-y += lib/div64.o
 EXT_COBJ-y += lib/string.o
 EXT_COBJ-y += lib/time.o
 EXT_COBJ-y += lib/vsprintf.o
+EXT_COBJ-y += lib/charset.o
 EXT_SOBJ-$(CONFIG_PPC) += arch/powerpc/lib/ppcstring.o
 ifeq ($(ARCH),arm)
 EXT_SOBJ-$(CONFIG_USE_ARCH_MEMSET) += arch/arm/lib/memset.o
diff --git a/lib/vsprintf.c b/lib/vsprintf.c
index 874a2951f7..0678b49b01 100644
--- a/lib/vsprintf.c
+++ b/lib/vsprintf.c
@@ -17,6 +17,7 @@
 #include 
 
 #include 
+#include 
 
 #include 
 #define noinline __attribute__((noinline))
@@ -270,6 +271,26 @@ static char *string(char *buf, char *end, char *s, int 
field_width,
return buf;
 }
 
+static char *string16(char *buf, char *end, u16 *s, int field_width,
+   int precision, int flags)
+{
+   u16 *str = s ? s : (u16[]){'<','N','U','L','L','>','\0'};
+   int utf16_len = utf16_strnlen(str, precision);
+   u8 utf8[utf16_len * MAX_UTF8_PER_UTF16];
+   int utf8_len, i;
+
+   utf8_len = utf16_to_utf8(utf8, str, utf16_len) - utf8;
+
+   if (!(flags & LEFT))
+   while (utf8_len < field_width--)
+   ADDCH(buf, ' ');
+   for (i = 0; i < utf8_len; ++i)
+   ADDCH(buf, utf8[i]);
+   while (utf8_len < field_width--)
+   ADDCH(buf, ' ');
+   return buf;
+}
+
 #ifdef CONFIG_CMD_NET
 static const char hex_asc[] = "0123456789abcdef";
 #define hex_asc_lo(x)  hex_asc[((x) & 0x0f)]
@@ -528,8 +549,14 @@ repeat:
continue;
 
case 's':
-   str = string(str, end, va_arg(args, char *),
-field_width, precision, flags);
+   if (CONFIG_IS_ENABLED(CC_SHORT_WCHAR) &&
+   qualifier == 'l') {
+   str = string16(str, end, va_arg(args, u16 *),
+  field_width, precision, flags);
+   } else {
+   str = string(str, end, va_arg(args, char *),
+field_width, precision, flags);
+   }
continue;
 
case 'p':
-- 
2.13.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot