Prefix internal property.

Author: davidlehn

lib/IdentifierIssuer.js

@@ -14,9 +14,9 @@ module.exports = class IdentifierIssuer {
    */
   constructor(prefix, existing = {refs: 0, map: new Map()}, counter = 0) {
     this.prefix = prefix;
-    this.existing = existing;
+    this._existing = existing;
     // add ref to shared map
-    this.existing.refs++;
+    this._existing.refs++;
     this.counter = counter;
   }
 
@@ -26,8 +26,8 @@ module.exports = class IdentifierIssuer {
    * @return a copy of this IdentifierIssuer.
    */
   clone() {
-    const {prefix, existing, counter} = this;
-    return new IdentifierIssuer(prefix, existing, counter);
+    const {prefix, _existing, counter} = this;
+    return new IdentifierIssuer(prefix, _existing, counter);
   }
 
   /**
@@ -40,7 +40,7 @@ module.exports = class IdentifierIssuer {
    */
   getId(old) {
     // return existing old identifier
-    const existing = old && this.existing.map.get(old);
+    const existing = old && this._existing.map.get(old);
     if(existing) {
       return existing;
     }
@@ -51,7 +51,7 @@ module.exports = class IdentifierIssuer {
 
     // save mapping
     if(old) {
-      if(this.existing.refs > 1) {
+      if(this._existing.refs > 1) {
         // copy-on-write shared map
         // TODO: improve copy-on-write reference handling
         //   - current code handles copying the 'existing' maps when it is
@@ -63,14 +63,14 @@ module.exports = class IdentifierIssuer {
         //   - this won't result in errors, only extra copies if a child does
         //     not do an update, is done, and a parent then does an update
         // unref shared map
-        this.existing.refs--;
+        this._existing.refs--;
         // copy to new map
-        this.existing = {
+        this._existing = {
           refs: 1,
-          map: new Map(this.existing.map)
+          map: new Map(this._existing.map)
         };
       }
-      this.existing.map.set(old, identifier);
+      this._existing.map.set(old, identifier);
     }
 
     return identifier;
@@ -86,7 +86,7 @@ module.exports = class IdentifierIssuer {
    *   false if not.
    */
   hasId(old) {
-    return this.existing.map.has(old);
+    return this._existing.map.has(old);
   }
 
   /**
@@ -96,6 +96,6 @@ module.exports = class IdentifierIssuer {
    * @return the list of old IDs that has been issued new IDs in order.
    */
   getOldIds() {
-    return [...this.existing.map.keys()];
+    return [...this._existing.map.keys()];
   }
 };