Details
Details
- Reviewers
cjdb • Quuxplusone curdeius - Group Reviewers
Restricted Project - Commits
- rGa85b1320b656: [libc++] Improve make_string test support.
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Sure; although here are some non-blockers:
- the repeated comment is maybe overkill
- the existing code's use of implicit conversion operators (instead of say an .as_chars<CharT>() getter) is just horrible and I'd love to see a PR to fix it throughout the test suite
- I wonder if you're going to need MAKE_STRING_VIEW next
Comment Actions
Thanks for the reviews. I'll update the comment before landing.
I did quite a bit more on std::format than what's up for review. I haven't had the need for MAKE_STRING_VIEW so I don't foresee I'll ever need it :-)
I think you've a good point about the implicit conversion. I'll add it to my todo list.