[libc-commits] [PATCH] D141162: [libc] add internal string class

[Siva Chandra via Phabricator via libc-commits]

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