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

Conversation

rainers
Copy link
Member

@rainers rainers commented Sep 21, 2013

http://d.puremagic.com/issues/show_bug.cgi?id=6045
http://d.puremagic.com/issues/show_bug.cgi?id=10027

allows function names as part of a qualified name, removing return type and function attributes. The compiler does this as well with PRETTY_FUNCTION, but also removes function arguments which probably is too much.

This needs dlang/phobos#1592 to pass unittests.

@rainers
Copy link
Member Author

rainers commented Sep 21, 2013

Can we tell the auto tester to merge a combination of branches from dmd/druntime/phobos for testing?
If not, what is the established work flow if phobos expects druntime to do something stupid?

@ghost
Copy link

ghost commented Sep 21, 2013

We can't do this with the tester now, but alternatively you could make a pull for phobos which simply comments out those two asserts (this one and this one) and then this druntime pull can be properly tested and merged, followed by testing and merging of dlang/phobos#1592

@rainers
Copy link
Member Author

rainers commented Sep 21, 2013

Thanks. I have just disabled the unittest in the phobos pull request, so it can be merged and still documents desired behaviour. Afer this pull is accepted, "version(none)" can be removed. (Another pull request will probably only have merge conflicts.)

@ghost
Copy link

ghost commented Sep 21, 2013

Ok merged it, let's wait for the results on this one now.

ghost pushed a commit that referenced this pull request Sep 21, 2013
fix issues 10277 & 6045:improve demangling for function local symbols
@ghost ghost merged commit 64ae912 into dlang:master Sep 21, 2013
Biotronic pushed a commit to Biotronic/phobos that referenced this pull request Oct 30, 2013
This pull request was closed.
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.

1 participant