[PATCH] D41017: Fix test/msan/iconv.cc on NetBSD

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 8 16:41:34 PST 2017


krytarowski added inline comments.


================
Comment at: test/msan/iconv.cc:19
+#if defined(__FreeBSD__) || defined(__NetBSD__)
+  // Due to old POSIX mistake some OSes expect 2nd argument of iconv() to be of type 'const char**'.
   const char *inbuf = inbuf_;
----------------
vitalybuka wrote:
> line is too long
I will commit it as `Some OSes expect the 2nd argument of iconv(3) to be of type const char **`


Repository:
  rL LLVM

https://reviews.llvm.org/D41017





More information about the llvm-commits mailing list