[libc-commits] [libc] [libc] Support %lc in printf (PR #169983)

[Alexey Samsonov via libc-commits]

================
@@ -33,7 +41,27 @@ LIBC_INLINE int convert_char(Writer<write_mode> *writer,
     RET_IF_RESULT_NEGATIVE(writer->write(' ', padding_spaces));
   }
 
-  RET_IF_RESULT_NEGATIVE(writer->write(c));
+#ifndef LIBC_COPT_PRINTF_DISABLE_WIDE
+  if (to_conv.length_modifier == LengthModifier::l) {
+    wint_t wi = static_cast<wint_t>(to_conv.conv_val_raw);
+
+    if (wi == WEOF) {
+      return -1;
----------------
vonosmas wrote:

Why are you returning blanket -1 instead of a more precise error? E.g. current printf engine has a dedicated INT_CONVERSION_ERROR. Should you return a proper value and check that the errno will be set to what it needs to?

https://github.com/llvm/llvm-project/pull/169983