Minor documentation and coding style update

Author: blu3mania

README.md

@@ -179,6 +179,7 @@ Registry class also provides the normal registry operations:
   corresponding JavaScript value types.
 
 - disableLogging()
+
   By default, methods in this package may log warnings and errors to console. If it's not desired, call this
   method to turn off logging. Note, it does not turn off the warnings logged in finalizers when opened keys
   and monitor tokens are not properly closed/stopped, because they indicate wrong usage of this package.

lib/windows-registry.js

@@ -1,7 +1,7 @@
 'use strict';
 
-const ref = require('ref-napi');
 const ffi = require('ffi-napi');
+const ref = require('ref-napi');
 
 const {
     error,
@@ -296,23 +296,23 @@ const EventApi = ffi.Library('kernel32', {
 });
 
 /** Finalizer for RegistryKey to ensure the underlying registry key handle is closed. */
-const RegistryKeyFinalizer = new FinalizationRegistry(registryKeyData => {
+const registryKeyFinalizer = new FinalizationRegistry(registryKeyData => {
     if (registryKeyData.handle !== null) {
         warning(`RegistryKey ${registryKeyData.path} is not properly closed! You should call RegistryKey.close() or Registry.closeKey() to close a key when it is no longer needed!`);
         Registry.instance.closeKeyInternal(registryKeyData);
     }
 });
 
 /** Finalizer for MonitoredRegistryKey to ensure the underlying event handle is closed. */
-const MonitoredRegistryKeyFinalizer = new FinalizationRegistry(monitorData => {
+const monitoredRegistryKeyFinalizer = new FinalizationRegistry(monitorData => {
     if (monitorData.waitHandle !== null) {
         warning(`MonitoredRegistryKey ${registryKeyData.path} is not properly stopped! You should call Registry.stopMonitor() to stop monitoring a key when it is no longer needed!`);
         Registry.instance.stopMonitoredKeyInternal(monitorData);
     }
 });
 
 /** Finalizer for MonitorToken to ensure the callback is unregistered. */
-const MonitorTokenFinalizer = new FinalizationRegistry(tokenData => {
+const monitorTokenFinalizer = new FinalizationRegistry(tokenData => {
     if (!tokenData.calback !== null) {
         warning(`MonitorToken on key ${tokenData.path} is not properly removed! You should call MonitorToken.stop() or Registry.stopMonitor() to stop monitoring a key when it is no longer needed!`);
         Registry.instance.stopMonitorInternal(tokenData);
@@ -356,7 +356,7 @@ class RegistryKey {
         this.open();
 
         // Register this key in finalizer so we can be alerted if it's not properly closed
-        RegistryKeyFinalizer.register(this, this.keyData);
+        registryKeyFinalizer.register(this, this.keyData);
     }
 
     /**
@@ -740,7 +740,7 @@ class MonitoredRegistryKey extends RegistryKey {
         this.start();
 
         // Register this key in finalizer so we can be alerted if it's not properly stopped
-        MonitoredRegistryKeyFinalizer.register(this, this.monitorData);
+        monitoredRegistryKeyFinalizer.register(this, this.monitorData);
     }
 
     /**
@@ -898,7 +898,7 @@ class MonitorToken {
         };
 
         // Register this token in finalizer so we can be alerted if it's not properly stopped
-        MonitorTokenFinalizer.register(this, this.tokenData);
+        monitorTokenFinalizer.register(this, this.tokenData);
     }
 
     /**

lib/windows-system-error-text.js

@@ -1,7 +1,7 @@
 'use strict';
 
-const ref = require('ref-napi');
 const ffi = require('ffi-napi');
+const ref = require('ref-napi');
 
 const { fromNullTerminatedWString } = require('./wstring.js');