Remove len property

dingosky · dingosky · commit ad0d7237b46d · 2017-08-10T07:50:03.000-07:00
diff --git a/lib/charSet.js b/lib/charSet.js
@@ -4,9 +4,8 @@ class CharSet {
   constructor(chars, ndxFn) {
     this.chars = chars
     this.ndxFn = ndxFn
-    this.len = chars.length
-    this.entropyPerChar = Math.floor(Math.log2(this.len))
-    if (this.entropyPerChar != Math.log2(this.len)) {
+    this.entropyPerChar = Math.floor(Math.log2(chars.length))
+    if (this.entropyPerChar != Math.log2(chars.length)) {
       throw new Error('EntropyString only supports CharSets with a power of 2 characters')
     }
     this.charsPerChunk = lcm(this.entropyPerChar, 8) / this.entropyPerChar
@@ -15,7 +14,7 @@ class CharSet {
   use(chars) {
     const len = chars.length
     // Ensure correct number of characters
-    if (len != this.len) {
+    if (len != this.chars.length) {
       throw new Error('Invalid character count')
     }
     // Ensure no repeated characters
