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

Issue 15186 - Emit better diagnostic for C++ member lookup operators #5190

Merged
merged 1 commit into from
Oct 11, 2015

Conversation

AndrejMitrovic
Copy link
Contributor

@AndrejMitrovic
Copy link
Contributor Author

That's a very bizarre segfault..

make[1]: *** [unittest/std/concurrency.run] Segmentation fault (core dumped)

Can't reproduce it locally, although I have to build with MODEL=64 and not 32-bit like the autotester (32-bit builds fail for me).

I'll try to force a re-test.

@AndrejMitrovic AndrejMitrovic force-pushed the fix-15186 branch 2 times, most recently from a42d240 to 4a7fbdc Compare October 11, 2015 18:45
@andralex
Copy link
Member

@AndrejMitrovic I think I've seen that before. Are you running inside a VM?

@andralex
Copy link
Member

Auto-merge toggled on

@andralex
Copy link
Member

Nice - thanks!

@AndrejMitrovic
Copy link
Contributor Author

@AndrejMitrovic I think I've seen that before. Are you running inside a VM?

The 32-bit build fail? Yeah.

P.S. I've had problems with my account and I'm no longer on the team list. Any way you could add me back so I could start merging some pulls? Yeah, I'm back in the game. :)

andralex added a commit that referenced this pull request Oct 11, 2015
Issue 15186 - Emit better diagnostic for C++ member lookup operators
@andralex andralex merged commit c5a0636 into dlang:master Oct 11, 2015
@AndrejMitrovic AndrejMitrovic deleted the fix-15186 branch October 11, 2015 20:29
@yebblies
Copy link
Member

Yeah, I'm back in the game. :)

Welcome back!

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