Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

no viable conversion from 'Local<v8::Context>' to 'v8::Isolate *' #117

Closed
jacobsologub opened this issue Aug 21, 2019 · 4 comments
Closed

Comments

@jacobsologub
Copy link
Contributor

Getting the following error when including the library. c++17 branch

#include <v8pp/module.hpp>
#include <v8pp/class.hpp>
In file included from ../../vendor/v8pp/v8pp/module.hpp:14:
In file included from ../../vendor/v8pp/v8pp/function.hpp:16:
In file included from ../../vendor/v8pp/v8pp/call_from_v8.hpp:17:
../../vendor/v8pp/v8pp/convert.hpp:145:30: error: no viable conversion from 'Local<v8::Context>' to 'v8::Isolate *'
                return value->BooleanValue(isolate->GetCurrentContext()).FromJust();
@jacobsologub
Copy link
Contributor Author

updating...

return value->BooleanValue(isolate).FromJust();

now getting this:

In file included from ../../vendor/v8pp/v8pp/module.hpp:14:
In file included from ../../vendor/v8pp/v8pp/function.hpp:16:
In file included from ../../vendor/v8pp/v8pp/call_from_v8.hpp:17:
../../vendor/v8pp/v8pp/convert.hpp:145:38: error: member reference base type 'bool' is not a structure or union
                return value->BooleanValue(isolate).FromJust();
                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~

@jacobsologub
Copy link
Contributor Author

macOS, Xcode 10.2.1 (10E1001)

@pmed
Copy link
Owner

pmed commented Aug 21, 2019 via email

@jacobsologub
Copy link
Contributor Author

jacobsologub commented Aug 21, 2019

@pmed I'm using the latest 7.8.151

local fix:

return value->BooleanValue(isolate);

pmed pushed a commit that referenced this issue Nov 10, 2019
…<bool>::from_v8

v8 8.x is around the corner and the current check fails.
Addition to PR #123 to fix issue #117

4ce49ef
sankayop added a commit to sankayop/core that referenced this issue Jan 25, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants