New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Yet more cherry-picks for c++20 #4227
Conversation
... suggesting that the more common plain "object type" means something different.
to match [counted.iter.cust]
I might perhaps like to drop the "wrap synopsis in namespace" commit, just because it's a bit large. |
It's large, but if you ignore whitespace ( --- a/source/strings.tex
+++ b/source/strings.tex
@@ -4020,9 +4020,10 @@ are available when \tcode{<string_view>} is included.
\indexlibrarymember{size_type}{basic_string_view}%
\indexlibrarymember{difference_type}{basic_string_view}%
\begin{codeblock}
-template<class charT, class traits = char_traits<charT>>
-class basic_string_view {
-public:
+namespace std {
+ template<class charT, class traits = char_traits<charT>>
+ class basic_string_view {
+ public:
// types
using traits_type = traits;
using value_type = charT;
@@ -4125,14 +4126,15 @@ public:
size_type n) const;
constexpr size_type find_last_not_of(const charT* s, size_type pos = npos) const;
-private:
+ private:
const_pointer data_; // \expos
size_type size_; // \expos
-};
+ };
-// \ref{string.view.deduct}, deduction guide
-template<class It, class End>
+ // \ref{string.view.deduct}, deduction guide
+ template<class It, class End>
basic_string_view(It, End) -> basic_string_view<iter_value_t<It>>;
+}
\end{codeblock}
\pnum |
After this speculative PR, the remaining unused commits would be as follows:
|
I added dd52b47 to this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good assuming someone has checked there are no unexpected changes in the big strings.tex diff.
@jwakely posted a diff that disregards whitespace and shows that this is really just adding the namespace and indenting everything inside. |
@jwakely, @CaseyCarter your input is most welcome. See also #4220 (comment) for a complete list of candidate commits.