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
Nan::Maybe<T> ToChecked not implemented for Node.js 6.x #851
Comments
1 task
kkoopa
pushed a commit
that referenced
this issue
May 16, 2019
* Add missing methods to Nan::Maybe<T> Nan::Maybe<T> was an alias for v8::Maybe<T> with most Node.js versions but that's problematic because v8::Maybe<T> has grown new methods over time that are missing in older versions. From now on we use our custom Maybe<T> implementation with all Node.js versions. This commit also adds the following missing methods: * Maybe<T>::Check() const * Maybe<T>::To(T*) const * Maybe<T>::ToChecked() const Fixes: #851 * Fix cpplint warnings. The code base is now `make lint`-clean again.
@kkoopa would love it if you could tag a release with this in ❤️ would like to get my PR to node-canvas moving forward 🐎 |
matburnham
added a commit
to matburnham/node-osmium
that referenced
this issue
Jul 28, 2020
to build on Node 6.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm getting the following error when I fixing a warning that told me the return value of
Get
was unchecked.Automattic/node-canvas#1415 • Travis CI jobs/526851510#L638
Is this something that should be added to
nan_maybe_pre_43_inl.h
?The text was updated successfully, but these errors were encountered: