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
Use \placeholder for alias templates in container deduction guides #1965
Conversation
That's very nice! |
source/containers.tex
Outdated
\begin{codeblock} | ||
template<class InputIterator> | ||
using @\placeholder{iter-value-type}@ = | ||
typename iterator_traits<InputIterator>::value_type>; // \expos |
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.
Stray >
at the end.
source/containers.tex
Outdated
@@ -5797,13 +5797,16 @@ | |||
|
|||
\begin{codeblock} | |||
template<class InputIterator> | |||
using iter_key_t = remove_const_t< | |||
using @\placeholder{iter-value-type}@ = | |||
typename iterator_traits<InputIterator>::value_type>; // \expos |
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.
Here too.
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.
fixed both of these, thanks.
…plates Also rename iter_val_t to iter-mapped-type and then add iter-value-type for value_type. Fixes cplusplus#1523
I've stolen your commits and altered them. (I adjusted the horizontal spacing, removing extraneous line breaks, harmonizing the comment column, etc.) |
Rename the alias templates and style as placeholders:
iter_key_t
-> iter-key-typeiter_val_t
-> iter-mapped-typeiter_to_alloc_t
-> iter-to-alloc-typeAdd iter-value-type for use in sequence containers and sets.
Fixes #1523