Skip to content

Issue 14818 - Unhelpful "does not match template overload set" error#4826

Merged
yebblies merged 1 commit intodlang:masterfrom
9rnsr:fix14818
Aug 30, 2015
Merged

Issue 14818 - Unhelpful "does not match template overload set" error#4826
yebblies merged 1 commit intodlang:masterfrom
9rnsr:fix14818

Conversation

@9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Jul 21, 2015

https://issues.dlang.org/show_bug.cgi?id=14818

The created OverloadSet object should have the searched identifier.

Updated 2015/07/22:
Also, the overload objects OverDeclaration and OverloadSet that is generated by AliasDeclaration should have the alias identifier.

@9rnsr 9rnsr force-pushed the fix14818 branch 2 times, most recently from e3c42b6 to 85590e7 Compare July 22, 2015 13:55
The returned OverloadSet object from Dsymbol::search should have the searched identifier.
Also, the overload objects OverDeclaration and OverloadSet that is generated by AliasDeclaration should have the alias identifier.
@9rnsr
Copy link
Contributor Author

9rnsr commented Aug 30, 2015

Rebased on master, and ready for ddmd.

@yebblies
Copy link
Contributor

Auto-merge toggled on

@9rnsr
Copy link
Contributor Author

9rnsr commented Aug 30, 2015

Thx!

yebblies added a commit that referenced this pull request Aug 30, 2015
Issue 14818 - Unhelpful "does not match template overload set" error
@yebblies yebblies merged commit 6f90c0d into dlang:master Aug 30, 2015
@9rnsr 9rnsr deleted the fix14818 branch August 30, 2015 13:35
@MartinNowak
Copy link
Member

Rebased on master, and ready for ddmd.

Could you use the method described here?

@9rnsr
Copy link
Contributor Author

9rnsr commented Sep 2, 2015

@MartinNowak Of course I know that, but that method did create unmaintainable entangled graphs in my local repository. So finally I decided to rebase all PR branches by hand.

@MartinNowak
Copy link
Member

That's a pity, it works very well for me.

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.

3 participants