Closed pepeiborra closed 7 years ago
Hm, it seems to me that ffiExport'
is beginning to duplicate the API in Foreign.JavaScript
. I think it's better to export the function liftJSWindow
right away.
Also, could you put both functions in an extra section "Internals"? This makes it more clear that they are stopgap measures.
Done as requested.
Looks good now. Thanks!
It looks like this PR was merged a couple of months before the 0.8.0.1 release, but it was not included there. Was that an oversight?
Adding new functions would require a 0.8.1.0 release. Our co-maintainer Simon Jakobi made the 0.8.0.1 release (thanks!) mainly to ensure compatibility with GHC 8.2.
Makes sense!
Next time I need to bump a dependency version, I'll look what kind of unreleased changes are lying around. If there are any, I'll ask @HeinrichApfelmus if he'd mind me releasing them too.
Ok, sounds good, thanks!
As discussed, this will enable easier management of custom JSObject lifetimes, as illustrated by the new version of ffiExport.