Module is not Context-Aware. Creates module registration error with node js worker threads about mmap-io HOT 5 CLOSED

I've discovered this has to do with making the module context-aware. There is an open issue on nodejs detailing some other affected modules as well as some solutions.

nodejs/node#21783 (comment)

Some more documentation:
https://nodejs.org/docs/latest/api/addons.html#addons_context_aware_addons

I'm attempting a fix but have not found any luck thus far. My C++ skills are very novice so this stuff is going over my head and I'm really just trying to emulate what others have done to migrate their modules (like here: https://github.com/schroffl/node-lzo/pull/11/files)

from mmap-io.

@Ethan-Arrowood — I quickly looked at the linked commentary, and did some very minor edits, not sure if the macro use changes does anything at all for you, but please try it out.

from mmap-io.

No follow up, so I'm closing. Re-open if the problem subsists.

from mmap-io.

I'm receiving this deprecation message when using mmap-io in Electron:

Electron: Loading non-context-aware native module in renderer. This is deprecated

This is fixed in master thanks to this change daeff71

@ozra could you release a new version?

from mmap-io.

@ozra New release version with daeff71 will be great!

from mmap-io.