Skip to content

Issue 5718 - Can't demangle symbol defined inside unittest block #414

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 1, 2012

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Feb 1, 2012

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

std.traits.mangledName should return valid mangled name against symbols in function.

std.traits.mangledName should return valid mangled name against symbols in function.
else
{
enum string mangledName = sth[0].mangleof;
}
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just as an aside, I'm certain that I wrote the original version of that code. It was originally called "mangleNameOf", and was published on the ng and in DDL project on dsource. Somebody copied it verbatim and republished it another dsource project with my name removed and most of the comments stripped out, but with the function names unchanged. I think this code is based on that. This code still doesn't include the comments explaining why it works, and it really needs to. It took a lot of creativity to come up with, it's really not obvious.

@WalterBright
Copy link
Member

Don, please add your explanatory comments & credit back in (in a separate pull, as I'm pulling this one now).

WalterBright added a commit that referenced this pull request Feb 1, 2012
Issue 5718 - Can't demangle symbol defined inside unittest block
@WalterBright WalterBright merged commit ac88792 into dlang:master Feb 1, 2012
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