phobos icon indicating copy to clipboard operation
phobos copied to clipboard

co-locate overloads of std.algorithm.mutation.swap()

Open WalterBright opened this issue 3 months ago • 4 comments

The two overloads are widely separated. This puts them together for easier study. The changes are mostly indenting.

Also has its own error message rather than the compiler generated one.

WalterBright avatar Mar 18 '24 16:03 WalterBright

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#8951"

dlang-bot avatar Mar 18 '24 16:03 dlang-bot

Gave up on the own message and the merge. Just co-located them.

WalterBright avatar Mar 18 '24 17:03 WalterBright

dstyle fails with no message.

WalterBright avatar Mar 18 '24 22:03 WalterBright

Error: The command was interrupted by a signal: signal: terminated

It looks like DScanner was stuck in a loop for an hour and got terminated

dkorpel avatar Mar 19 '24 12:03 dkorpel