P2278R4: `views::as_const` by JMazurkiewicz · Pull Request #3234 · microsoft/STL · GitHub
Skip to content

P2278R4: views::as_const#3234

Merged
StephanTLavavej merged 8 commits into
microsoft:mainfrom
JMazurkiewicz:as_const_view
Dec 15, 2022
Merged

P2278R4: views::as_const#3234
StephanTLavavej merged 8 commits into
microsoft:mainfrom
JMazurkiewicz:as_const_view

Conversation

@JMazurkiewicz

@JMazurkiewicz JMazurkiewicz commented Nov 16, 2022

Copy link
Copy Markdown
Contributor

Closes #2918 - "cbegin should always return a constant iterator".

@StephanTLavavej

Copy link
Copy Markdown
Member

@JMazurkiewicz

Copy link
Copy Markdown
Contributor Author

And... it's an ICE 😤

@barcharcraz

Copy link
Copy Markdown
Contributor

Do you happen to have a reduced case for this ICE? If not don't worry about it, I'll reduce it and push a transition comment.

@JMazurkiewicz

Copy link
Copy Markdown
Contributor Author

Do you happen to have a reduced case for this ICE?

Nope

@barcharcraz

Copy link
Copy Markdown
Contributor

Filed the frontend bug internally as VSO-1692469. I have not pushed a transition comment as I think the workaround improves the code anyway, so I don't think we need to come back to it when this bug is fixed.

Comment thread tests/std/tests/P2278R4_views_as_const/test.cpp
@barcharcraz barcharcraz removed their assignment Dec 7, 2022
@StephanTLavavej StephanTLavavej self-assigned this Dec 7, 2022
Comment thread tests/std/tests/P2278R4_views_as_const/test.cpp
Comment thread tests/std/tests/P2278R4_views_as_const/test.cpp Outdated
Comment thread tests/std/tests/P2278R4_views_as_const/test.cpp Outdated
Comment thread tests/std/tests/P2278R4_views_as_const/test.cpp Outdated
@StephanTLavavej StephanTLavavej removed their assignment Dec 10, 2022
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks, this looks fantastic! 😻 I pushed changes for 3 incredibly tiny nitpicks in the test code.

@StephanTLavavej StephanTLavavej self-assigned this Dec 15, 2022
@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit 032120d into microsoft:main Dec 15, 2022
@StephanTLavavej

Copy link
Copy Markdown
Member

@JMazurkiewicz JMazurkiewicz deleted the as_const_view branch December 15, 2022 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cxx23 C++23 feature ranges C++20/23 ranges

Projects

None yet

Development

Successfully merging this pull request may close these issues.

P2278R4 cbegin Should Always Return A Constant Iterator

4 participants