Skip to content

Merge remote-tracking branch 'upstream/stable' into merge_stable#4889

Merged
MartinNowak merged 11 commits intodlang:masterfrom
MartinNowak:merge_stable
Nov 20, 2016
Merged

Merge remote-tracking branch 'upstream/stable' into merge_stable#4889
MartinNowak merged 11 commits intodlang:masterfrom
MartinNowak:merge_stable

Conversation

@MartinNowak
Copy link
Member

Conflicts:
std/traits.d

CyberShadow and others added 11 commits October 13, 2016 08:26
- workaround scope escape error to get default parameter
- would require a working `return scope` for a proper implementation
reduce test template bloat
fix Issue 16582 - ParameterDefaults fails w/ scope parameter
Also put the signature of `isImplicitlyConvertible` in a comment to make the example more clear.
❗️❗️❗️Implicit cast to const slices in case of underlaying range is a pointer.
Resurrect ApplyLeft and ApplyRight's documentation
@dlang-bot
Copy link
Contributor

Fix Bugzilla Description
16514 std.socket methods are const, and thus cannot be overriden for SSLSocket, for example
16582 [REG2.072.0-b1] ParamterDefaultValueTuple fails to compile for scope paramters

@MartinNowak
Copy link
Member Author

Why is this stuck?

@schveiguy
Copy link
Member

@MartinNowak Looks fine, what does the "conflicts" comment at the top mean?

@MartinNowak
Copy link
Member Author

@MartinNowak Looks fine, what does the "conflicts" comment at the top mean?

If you merge on the command line, that's the commit message git creates when merge conflicts occur.
In this case I had to resolve a merge conflict in std.traits, where ParameterDefaults was changed in stable #4864 and master #4862.

@andralex
Copy link
Member

@MartinNowak so what do we do here? Should I just merge?

@MartinNowak
Copy link
Member Author

Yes, I'll just merge it.

The resolution of the merge conflict is part of the merge commit (as always), the conflict comment is just a git remark for the history.

@MartinNowak
Copy link
Member Author

Auto-merge toggled on

@MartinNowak MartinNowak merged commit 26a6857 into dlang:master Nov 20, 2016
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.

9 participants