Skip to content

Rewrite std.meta.Stride using AliasAssign#7758

Closed
WalterBright wants to merge 1 commit intodlang:masterfrom
WalterBright:AliasAssignStride
Closed

Rewrite std.meta.Stride using AliasAssign#7758
WalterBright wants to merge 1 commit intodlang:masterfrom
WalterBright:AliasAssignStride

Conversation

@WalterBright
Copy link
Member

Also added shortcut when stepSize is 1.

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @WalterBright!

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 run digger -- build "master + phobos#7758"

@WalterBright
Copy link
Member Author

@razvan7 the failures are the same as the ones for #7756

Was something pulled that broke the Phobos tests?

@Geod24
Copy link
Member

Geod24 commented Jan 22, 2021

Was something pulled that broke the Phobos tests?

Master was green 4 hours ago, so I doubt so: https://github.com/dlang/phobos/commits/master

@pbackus
Copy link
Contributor

pbackus commented Jan 28, 2021

I saw similar failures recently in #7702. They went away after a rebase.

@RazvanN7
Copy link
Collaborator

RazvanN7 commented Mar 8, 2021

@WalterBright Please rebase.

@RazvanN7
Copy link
Collaborator

ping @WalterBright . Where are we with this?

@RazvanN7
Copy link
Collaborator

RazvanN7 commented May 6, 2021

Closing in favor of #8047 as it produces the same change but in less lines of code.

@RazvanN7 RazvanN7 closed this May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Comments