-
Notifications
You must be signed in to change notification settings - Fork 2
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
Moving to c++11? #47
Comments
This should be possible, I hope to have a look at it in the next week or so. |
libc++ is only supported on Mac OS X 10.7 and high
Seems to work, apart from one error:
It does mean dropping support for Mac OS X 10.5 and 10.6, but I have no idea if VDrift works on them any more anyway. |
Spotted with osx/clang c++11 mode. VDrift/vdrift-mac#47
That's a bug, luckily a trivial one. |
Now something isn't linking right:
|
Just did a quick grep for length(). This should fail in a few other places too. Replacing it with size() should do the job. |
Hmm, it might be a bit more involved. |
They are not guaranteed to be visible to the linker, can be inlined. Hit with osx/clang VDrift/vdrift-mac#47
I have the same problem in FreeBSD. I found that std::mem_fun_ref was deprecated in cplusplus.com reference , and I tested the example and it didn't work(get the same error). So maybe use std::mem_fn instead(I don't know how to use it) |
VDrift/vdrift@31d53b5 fixes it 😄 |
See VDrift/vdrift#136
The text was updated successfully, but these errors were encountered: