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

Revert "Decrease template instantiation in std.algorithm.searching : all & any" #6988

Merged
merged 1 commit into from
May 2, 2019

Conversation

n8sh
Copy link
Member

@n8sh n8sh commented Apr 29, 2019

Reverts #6960. Reason: foreach does not auto-decode string & wstring.

@dlang-bot
Copy link
Contributor

dlang-bot commented Apr 29, 2019

Thanks for your pull request, @n8sh!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub fetch digger
dub run digger -- build "stable + phobos#6988"

@wilzbach
Copy link
Member

Can we add a test case that fails now, but will be fixed by the revert?

@n8sh n8sh force-pushed the revert-6960-searchingAllAnyFewerTemplates branch from a333041 to 08e49dd Compare April 29, 2019 17:52
@n8sh
Copy link
Member Author

n8sh commented Apr 29, 2019

@wilzbach added tests.

Copy link
Member

@wilzbach wilzbach left a comment

Choose a reason for hiding this comment

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

Thanks!

@PetarKirov
Copy link
Member

@n8sh please rebase your branch onto stable, as otherwise the upcoming release won't include your fix. As a rule of thumb, if a bug exists in the stable branch, the bug fix should also be merged there.

@n8sh
Copy link
Member Author

n8sh commented May 1, 2019

@ZombineDev At the time this PR was submitted the bug didn't exist in stable. Time marched on.

@n8sh n8sh changed the base branch from master to stable May 1, 2019 15:12
@n8sh n8sh force-pushed the revert-6960-searchingAllAnyFewerTemplates branch from 08e49dd to 384320c Compare May 1, 2019 15:19
@n8sh n8sh force-pushed the revert-6960-searchingAllAnyFewerTemplates branch from 384320c to d410281 Compare May 1, 2019 19:46
@dlang-bot dlang-bot merged commit 7a4ac98 into stable May 2, 2019
@dnadlinger dnadlinger deleted the revert-6960-searchingAllAnyFewerTemplates branch May 2, 2019 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants