Avoid exposition of _eventHandlers / _eventEmitter

index.js

@@ -7,9 +7,10 @@ const ReactNative = require('react-native');
 const NativeEventEmitter = ReactNative.NativeEventEmitter;
 const RNDevMenu = ReactNative.NativeModules.RNDevMenu;
 
+let emitter: NativeEventEmitter | void;
+let handlers: Map<string, () => any> | void;
+
 type RNDevMenuModule = {
-  _eventHandlers?: Map<string, () => any>,
-  _eventEmitter?: NativeEventEmitter,
   addItem: (name: string, handler: () => any) => Promise<void>
 };
 
@@ -19,25 +20,21 @@ let DevMenu: RNDevMenuModule = {
       return Promise.resolve();
     }
 
-    if (!this._eventHandlers) {
-      this._eventHandlers = new Map();
+    if (!emitter) {
+      emitter = new NativeEventEmitter(RNDevMenu);
+    }
+    if (!handlers) {
+      handlers = new Map();
     }
 
-    this._eventHandlers.set(name, handler);
-
-    if (!this._eventEmitter) {
-      this._eventEmitter = new NativeEventEmitter(RNDevMenu);
-
-      this._eventEmitter.addListener('customDevOptionTap', (name: string) => {
-        if (this._eventHandlers) {
-          const handler = this._eventHandlers.get(name);
+    handlers.set(name, handler);
 
-          if (handler) {
-            handler();
-          }
-        }
-      });
-    }
+    emitter.addListener('customDevOptionTap', (name: string) => {
+      if (handlers) {
+        const handler = handlers.get(name);
+        handler && handler();
+      }
+    });
 
     return RNDevMenu.addItem(name);
   }