[webkit-gtk] TextCodec.cpp lacks include <cstdio> when compiling with clang-7.0

[Adrian Perez de Castro]

On Wed, 22 May 2019 22:58:01 +0300, Denis Pronin <dannftk at yandex.ru> wrote:
Non-text part: text/html

> Actually, including <cstdio> would do because it declares 'using
> std::snprinf' in global space. Moreover, there is a common recomendation
> to use c++ wrappers starting from <c...> over c-headers ending with <.h>
> in c++ code. I would rather you followed this recomendation to not break
> the rule

The WebKit coding style [1] does not specify what to do in this case where
both a plain C (stdio.h) and C++ headers (cstdio) are available, so I agree
with Denis here that it is better to use the latter (cstdio)

Cheers,
—Adrián

---
[1] https://webkit.org/code-style-guidelines/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <http://lists.webkit.org/pipermail/webkit-gtk/attachments/20190523/dae6cacd/attachment.bin>