[libc-commits] [PATCH] D125929: [libc] add printf base 10 integer conversion

Siva Chandra via Phabricator via libc-commits libc-commits at lists.llvm.org
Tue Jun 7 12:04:37 PDT 2022

sivachandra added inline comments.

Comment at: libc/test/src/stdio/printf_core/converter_test.cpp:27
sivachandra wrote:
> I am probably missing something - how are these changes to string conversion tests related to integer conversion? Also, do we really want a separate string converter and writer for every test? If not, they can be static globals.
> Another point, which can be addressed separately - we should probably not be testing against internal API for a converter, writer etc. You can have a test for verifying that the plumbing works, but everything else should probably be tested using `sprintf`.
I forgot that `StringWriter` is stateful - ignore that part of the comment about needing a new string writer for every test.

  rG LLVM Github Monorepo



More information about the libc-commits mailing list