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

Test LWG-4105 ranges::ends_with's Returns misses difference casting #4897

Merged
merged 7 commits into from
Aug 25, 2024

Conversation

frederick-vs-ja
Copy link
Contributor

Closes #4765.

This PR confirms that the resolution is effectively implemented at first by testing two ranges with difference types being an integer type and an integer-class type repectively.

  • I'd like to add redifference_iterator/redifference_sentinel to <range_algorithm_support.hpp> because I think they will be useful for addressing many difference-type-related issues.
  • A new test file is created because the test coverage is unexpectedly heavy for compilation.

@frederick-vs-ja frederick-vs-ja requested a review from a team as a code owner August 18, 2024 12:42
@StephanTLavavej StephanTLavavej added LWG Library Working Group issue ranges C++20/23 ranges labels Aug 18, 2024
@StephanTLavavej StephanTLavavej self-assigned this Aug 18, 2024
@StephanTLavavej StephanTLavavej added the test Related to test code label Aug 19, 2024
tests/std/include/range_algorithm_support.hpp Show resolved Hide resolved
tests/std/include/range_algorithm_support.hpp Outdated Show resolved Hide resolved
tests/std/include/range_algorithm_support.hpp Outdated Show resolved Hide resolved
tests/std/include/range_algorithm_support.hpp Outdated Show resolved Hide resolved
tests/std/include/range_algorithm_support.hpp Outdated Show resolved Hide resolved
tests/std/include/range_algorithm_support.hpp Outdated Show resolved Hide resolved
tests/std/include/range_algorithm_support.hpp Show resolved Hide resolved
tests/std/include/range_algorithm_support.hpp Outdated Show resolved Hide resolved
@StephanTLavavej StephanTLavavej removed their assignment Aug 19, 2024
@StephanTLavavej

This comment was marked as resolved.

@StephanTLavavej StephanTLavavej self-assigned this Aug 25, 2024
@StephanTLavavej
Copy link
Member

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

@StephanTLavavej StephanTLavavej merged commit 20a1c9f into microsoft:main Aug 25, 2024
39 checks passed
@StephanTLavavej
Copy link
Member

Thanks for verifying this LWG issue! ✅ 💯 🎉

@frederick-vs-ja frederick-vs-ja deleted the lwg-4105 branch August 25, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LWG Library Working Group issue ranges C++20/23 ranges test Related to test code
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

LWG-4105 ranges::ends_with's Returns misses difference casting
3 participants