[libc-commits] [PATCH] D141162: [libc] add internal string class
Siva Chandra via Phabricator via libc-commits
libc-commits at lists.llvm.org
Fri Jan 6 16:28:21 PST 2023
sivachandra added inline comments.
================
Comment at: libc/src/__support/CPP/vector_string.h:24
+
+class VectorString {
+ static constexpr size_t INIT_BUFF_SIZE = 64;
----------------
michaelrj wrote:
> sivachandra wrote:
> > Shouldn't this be named `string`?
> I talked about it in my comment on the patch this was split out of but basically it has some differences from `std::string` that might warrant a different name.
Ah, sorry! I did not look at that patch yet.
Going by your explanation, what you have done previously is correct - Keep is `scanf` specific as we cannot add non-standard API in the `CPP` directory. Instead of `VectorString`, may be call it `CharVector`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141162/new/
https://reviews.llvm.org/D141162
More information about the libc-commits
mailing list