[vector.bool] Remove an extra close-paren. #4501
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In this vicinity I also noticed:
The copy-constructor and assignment-operators for both flavors of
vector
give the rhs argument the name "x
", which seems a bit silly; it could go unnamed if we wanted.vector<bool>
's move-constructor is not declarednoexcept
. This was deliberate; I don't really know why, though. N4258 madevector<T>
's move-ctor noexcept, and says briefly at the bottom: "No change in vector" (no rationale).The sequence containers and
set
declare constructors takinginitializer_list<T>
, whereas the maps andunordered_{multi,}set
declare constructors takinginitializer_list<value_type>
. (This is almost entirely consistent with the hypothesis that for simplicity we useT
wherever it's synonymous withvalue_type
; except thatunordered_{multi,}set
breaks that rule.)