Skip to content
This repository was archived by the owner on Oct 12, 2022. It is now read-only.
/ druntime Public archive

Add tests for class/struct/array comparisons#1784

Merged
MartinNowak merged 2 commits intodlang:masterfrom
CyberShadow:pull-20170304-141153
Mar 4, 2017
Merged

Add tests for class/struct/array comparisons#1784
MartinNowak merged 2 commits intodlang:masterfrom
CyberShadow:pull-20170304-141153

Conversation

@CyberShadow
Copy link
Member

PR #1781 is moving array comparisons to a new template; the initial implementation included changes which may have introduced performance and behavior regressions.

Add explicit tests for these cases to detect such regressions.

The introduction of the .gitignore rule lacked any description or
rationale (PR dlang#682), so I'm guessing this is to allow people to create
a local Makefile so that `make` works without needing to specify `-f
posix.mak`.

Fix this rule so that it doesn't apply to the actually versioned
Makefile files under the test directory.
@CyberShadow CyberShadow force-pushed the pull-20170304-141153 branch from f7b03fc to cdd0fee Compare March 4, 2017 14:21
PR dlang#1781 is moving array comparisons to a new template; the initial
implementation included changes which may have introduced performance
and behavior regressions.

Add explicit tests for these cases to detect such regressions.
@CyberShadow CyberShadow force-pushed the pull-20170304-141153 branch from cdd0fee to d388315 Compare March 4, 2017 14:23
@MartinNowak
Copy link
Member

Auto-merge toggled on

@andralex
Copy link
Member

andralex commented Mar 4, 2017

Thanks!

@CyberShadow
Copy link
Member Author

This would be one case where squashing is inappropriate :) Martin already enabled the old auto-merge.

@MartinNowak MartinNowak merged commit 7425809 into dlang:master Mar 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants