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

Non-const lvalue reference can't bind to temporary types. #21

Open
Sinoxan opened this issue Jul 15, 2015 · 3 comments
Open

Non-const lvalue reference can't bind to temporary types. #21

Sinoxan opened this issue Jul 15, 2015 · 3 comments

Comments

@Sinoxan
Copy link

Sinoxan commented Jul 15, 2015

Xcode gives quite a few errors where it complains...

Non-const lvalue reference to type 'el::Value' cannot bind to a temporary of type 'el::Value'

...such as in list_item.h with TakeValue function, I'm not sure if this simply needs something changing in Xcode to make the const explicit or if it's a feature that isn't supported in Xcode, just thought I'd point it out.

@benvanik
Copy link

Are you building with c++11 or c++14? Clang or gcc? (I am going to work
toward clang compatibility in the future, but don't know about gcc)
On Wed, Jul 15, 2015 at 13:21 Sinoxan [email protected] wrote:

Xcode gives quite a few errors where it complains...

Non-const lvalue reference to type 'el::Value' cannot bind to a temporary
of type 'el::Value'

...such as in list_item.h with TakeValue function, I'm not sure if this
simply needs something changing in Xcode to make the const explicit or if
it's a feature that isn't supported in Xcode, just thought I'd point it out.


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

@Sinoxan
Copy link
Author

Sinoxan commented Jul 15, 2015

I'm currently building with Clang's experimental C++17.

http://clang.llvm.org/cxx_status.html

@benvanik
Copy link

I've been meaning to try to get clang-on-windows going - that may be a good way to do migrations like this, as we can isolate the compiler differences from the bigger platform components. I know chrome has had luck with clang-on-windows (including supporting the full Windows SDK), but I'm not sure how much of that has been upstreamed out of chrome-land.

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

No branches or pull requests

2 participants