[libcxx-commits] [PATCH] D148693: [libc++] Set correct size at the end of growing std::string
Louis Dionne via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Jul 5 06:08:23 PDT 2023
ldionne accepted this revision.
ldionne added a comment.
LGTM with a suggestion to rephrase a comment.
================
Comment at: libcxx/include/string:2283-2285
+// __grow_by is deprecated because it does not set size. It not only may not update when size is changed,
+// but also not set size at all when string was short initially, leading to unpredictable size value.
+// It is not removed or changed to avoid breaking the ABI.
----------------
Just rewording, IMO this is easier to read. WDYT?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148693/new/
https://reviews.llvm.org/D148693
More information about the libcxx-commits
mailing list