You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
C++20 range adaptors such as filter_view, transform_view, take_view, take_while_view, drop_view, drop_while_view have no explicit keyword in their constructors.
However, C++23 range adaptors such as chunk_view, slide_view, and chunk_by_view all have the explicit keyword in their constructors.
We should make them consistent.
Also, in the current CTAD of all range adaptors, only join_view has the explicit keyword, do we need to remove this seemingly redundant explicit?
The text was updated successfully, but these errors were encountered:
Thanks for pointing out the second point.
Regarding the first point, I don't know how we should choose, because I don't see any observable benefit or harm from explicit.
filter_view
,transform_view
,take_view
,take_while_view
,drop_view
,drop_while_view
have noexplicit
keyword in their constructors.However, C++23 range adaptors such as
chunk_view
,slide_view
, andchunk_by_view
all have theexplicit
keyword in their constructors.We should make them consistent.
join_view
has theexplicit
keyword, do we need to remove this seemingly redundantexplicit
?The text was updated successfully, but these errors were encountered: