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

[associative] Remove redundant template arguments. #42

Merged
merged 1 commit into from Feb 2, 2013

Conversation

jwakely
Copy link
Member

@jwakely jwakely commented Jan 9, 2013

This removes redundant template arguments from [map.overview],
[multimap.overview], [set.overview] and [multiset.overview].

This is consistent with other member functions that rely on the injected
class name and I find it much easier to read the class synopses without
the clutter of the explicit template argument lists.
e.g. it's much easier to see that

map& operator=(map&& x);

is a move assignment operator rather than

map<Key,T,Compare,Allocator>&
  operator=(map<Key,T,Compare,Allocator>&& x);

This removes redundant template arguments from [map.overview],
[multimap.overview], [set.overview] and [multiset.overview].

This is consistent with other member functions that rely on the injected
class name and I find it much easier to read the class synopses without
the clutter of the explicit template argument lists.
@sdutoit
Copy link
Contributor

sdutoit commented Feb 2, 2013

This is great - thanks. I've done similar changes elsewhere in the past. I should look for more of these!

sdutoit added a commit that referenced this pull request Feb 2, 2013
[associative] Remove redundant template arguments. Thanks to Jonathan Wakely!
@sdutoit sdutoit merged commit 73ac1e7 into cplusplus:master Feb 2, 2013
@jwakely jwakely deleted the associative branch February 2, 2013 14:03
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

Successfully merging this pull request may close these issues.

None yet

2 participants