This adds the width estimation functions to the std-format-spec.
Implements parts of:
- P0645 Text Formatting
- P1868 width: clarifying units of width and precision in std::format
Paths
| Differential D103413
[libc++][format] Implement Unicode support. ClosedPublic Authored by Mordante on May 31 2021, 9:05 AM.
Details
Summary This adds the width estimation functions to the std-format-spec. Implements parts of:
Diff Detail
Event TimelineMordante added a child revision: D103425: [libc++][format] Adds string formatter..May 31 2021, 12:32 PM vitaut added inline comments.
This revision now requires changes to proceed.Jun 18 2021, 7:56 AM Mordante added inline comments.
Mordante marked 4 inline comments as done. Comment ActionsAddresses part of the review comments:
The other comments will be addressed later, I want to update the rest of the ser
Comment Actions Disable a test on Windows, that change was accidentally reverted in the last version. Mordante added inline comments.
Comment Actions Improve the readability of a long return statement, this found some issues in the return statement.
Comment Actions Thanks for the review!
Mordante marked 2 inline comments as done. Comment ActionsBased on review comments s/format_string_traits/string_alignment/
Mordante removed a parent revision: D103379: [libc++][format] Add a CMake Unicode option..Sep 4 2021, 2:48 AM Mordante removed a parent revision: D103368: [libc++][format] Adds parser std-format-spec..Sep 21 2021, 9:22 AM ldionne added inline comments.
This revision now requires changes to proceed.Sep 22 2021, 12:55 PM Comment Actions Thanks for the review. I'll update the patch after you land D110338.
This revision is now accepted and ready to land.Oct 1 2021, 1:50 PM Mordante removed a child revision: D103425: [libc++][format] Adds string formatter..Oct 2 2021, 2:55 AM Closed by commit rGac7031b2b2fa: [libc++][format] Implement Unicode support. (authored by Mordante). · Explain WhyOct 2 2021, 2:57 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 372870 libcxx/benchmarks/CMakeLists.txt
libcxx/benchmarks/std_format_spec_string_unicode.bench.cpp
libcxx/docs/Status/Cxx20Papers.csv
libcxx/include/__format/parser_std_format_spec.h
libcxx/test/libcxx/utilities/format/format.string/format.string.std/std_format_spec_string_unicode.pass.cpp
|
Can we make those changes as part of another review? Also, let's set CXX_STANDARD_REQUIRED YES instead -- all supported compilers should support C++20 now.