Skip to content
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

The deduction guides for unordered_map and unordered_multimap use an exposition-only helper that does not exist #1769

Closed
villevoutilainen opened this issue Oct 15, 2017 · 1 comment
Assignees
Milestone

Comments

@villevoutilainen
Copy link
Member

This is a simple typo fix from iter_value_t to iter_val_t.

In [unord.map.overview], edit as follows:

template<class InputIterator,
class Hash = hash<iter_key_t>,
class Pred = equal_to<iter_key_t>,
class Allocator = allocator<iter_to_alloc_t>>
unordered_map(InputIterator, InputIterator, typename see below
::size_type = see below ,
Hash = Hash(), Pred = Pred(), Allocator = Allocator())
-> unordered_map<iter_key_t,
iter_value_t, Hash, Pred,
Allocator>;

In [unord.multimap.overview], edit as follows:

template<class InputIterator,
class Hash = hash<iter_key_t>,
class Pred = equal_to<iter_key_t>,
class Allocator = allocator<iter_to_alloc_t>>
unordered_multimap(InputIterator, InputIterator,
typename see below ::size_type = see below ,
Hash = Hash(), Pred = Pred(), Allocator = Allocator())
-> unordered_multimap<iter_key_t,
iter_value_t, Hash, Pred,
Allocator>;

@villevoutilainen
Copy link
Member Author

Note that the LWG issue list maintainer thought this to be editorial. The changes are in normative text, but the typo fix seems obvious enough that this might be doable editorially.

jensmaurer added a commit to jensmaurer/draft that referenced this issue Oct 16, 2017
jensmaurer added a commit to jensmaurer/draft that referenced this issue Oct 16, 2017
@jensmaurer jensmaurer self-assigned this Oct 16, 2017
@jensmaurer jensmaurer modified the milestones: C++20, C++17 IS Oct 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants