Skip to content

Remove unused code from struct test_copy_if #2144

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

Merged
merged 2 commits into from
Mar 27, 2025

Conversation

SergeyKopienko
Copy link
Contributor

@SergeyKopienko SergeyKopienko commented Mar 26, 2025

In this PR we remove unused code - struct test_copy_if

@SergeyKopienko SergeyKopienko added the test Test only Change label Mar 26, 2025
@SergeyKopienko SergeyKopienko added this to the 2022.9.0 milestone Mar 26, 2025
@@ -60,8 +60,6 @@ struct test_copy_if

template <typename ExecutionPolicy, typename Iterator1, typename Iterator2>
void operator()(ExecutionPolicy&& /* exec */, Iterator1 /* first1 */, Iterator1 /* last1 */, Iterator2 /* first2 */) {
using value_type = typename ::std::iterator_traits<Iterator1>::value_type;
Copy link
Contributor

@MikeDvorskiy MikeDvorskiy Mar 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BTW, Is whether the struct test_copy_if used somewhere?
I could not find the usage in this test file...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed

@SergeyKopienko SergeyKopienko merged commit 3cf4229 into main Mar 27, 2025
19 checks passed
@SergeyKopienko SergeyKopienko deleted the dev/skopienko/remove_unused_code_in_test_copy_if branch March 27, 2025 08:52
timmiesmith pushed a commit that referenced this pull request Jun 9, 2025
* test/parallel_api/iterator/transform_iterator.pass.cpp - remove unused code in struct test_copy_if

* test/parallel_api/iterator/transform_iterator.pass.cpp - remove struct test_copy_if as unused
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Test only Change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants