Skip to content
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

Remove alias that is no longer needed in utility example. #8

Merged
merged 1 commit into from
Apr 15, 2015

Conversation

wilsonk
Copy link

@wilsonk wilsonk commented Apr 11, 2015

I suppose this may need to be reverted when the global namespace works again? Not sure, as make_pair still needs aliasing?

@Syniurge
Copy link
Owner

Hi Kelly,

Was the utility.d test not working anymore? By global namespace I meant "", not "std." and other "xyz.abc._"

"_" broke when I readded the decls inside extern "C" (LinkageSpec).

Are there errors while compiling utility.d? I can't test myself since I'm implementing the Clang module map file support to split the global namespace.

@wilsonk
Copy link
Author

wilsonk commented Apr 11, 2015

Hello Elie,

The error I was seeing with utility.d was as follows:

utility.d(11): Error: module cpp.std._ import 'pair' not found

which is fixed with the simple changes in the PR. I didn’t track down which commit caused the error, but I could if you like.

Thanks,

Kelly

From: Elie Morisse [mailto:[email protected]]
Sent: Saturday, April 11, 2015 8:14 AM
To: Syniurge/Calypso
Cc: wilsonk
Subject: Re: [Calypso] Remove alias that is no longer needed in utility example. (#8)

Hi Kelly,

Was the utility.d test not working anymore? By global namespace I meant "", not "std." and other "xyz.abc._"

"_" broke when I readded the decls inside extern "C" (LinkageSpec).

Are there errors while compiling utility.d? I can't test myself since I'm implementing the Clang module map file support to split the global namespace.


Reply to this email directly or view it on GitHub #8 (comment) .

Syniurge added a commit that referenced this pull request Apr 15, 2015
Remove alias that is no longer needed in utility example.
@Syniurge Syniurge merged commit af54d83 into Syniurge:master Apr 15, 2015
@wilsonk wilsonk mentioned this pull request May 9, 2015
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.

2 participants