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

Avoids error message wit g++ as C++ compiler #436

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dtolpin
Copy link

@dtolpin dtolpin commented Nov 7, 2015

GNU C++ issues an error on anonymous enum arguments used with typename in templates. -std=c++0x disables this error because the newer standard enables the usage. clang, the only other compiler option on supported architectures, works fine with the option (as well as without it).

@halgari
Copy link
Member

halgari commented Nov 8, 2015

I wouldn't be against putting this change inside ffi-infer and making a
default. That way it should allow us to work with all header files a bit
better.

Timothy

On Sat, Nov 7, 2015 at 1:10 PM, David Tolpin notifications@github.com
wrote:

GNU C++ issues an error on anonymous enum arguments used with typename in
templates. -std=c++0x disables this error because the newer standard
enables the usage. clang, the only other compiler option on supported

architectures, works fine with the option (as well as without it).

You can view, comment on, or merge this pull request online at:

#436
Commit Summary

  • added c++0x to avoid gnu c++ barking at me
  • Merge branch 'master' of ssh://github.com/dtolpin/pixie

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#436.

“One of the main causes of the fall of the Roman Empire was that–lacking
zero–they had no way to indicate successful termination of their C
programs.”
(Robert Firth)

@dtolpin
Copy link
Author

dtolpin commented Nov 8, 2015

Cool, then do not merge it yet, let me look at ffi-infer and suggest changes.

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