@woosh/meep-engine 2.48.22 → 2.49.0

package/src/core/cache/Cache.js

@@ -6,6 +6,7 @@ import { CacheElement } from "./CacheElement.js";
 import { collectIteratorValueToArray } from "../collection/IteratorUtils.js";
 
 /**
+ * Hash-based cache, uses LRU (least-recently-used) eviction policy
  * TODO validate hashes of held elements to keep them up to date. Keys are assumed to be immutable, but this assumption may be broken by users
  * @template Key, Value
  * @extends Map<Key,Value>
@@ -108,7 +109,8 @@ export class Cache {
     }
 
     /**
-     *
+     * Marks element as recently used
+     * Moves element to the end of the eviction queue
      * @param {CacheElement<Key,Value>} element
      * @private
      */
@@ -426,7 +428,7 @@ export class Cache {
          */
         const keys = [];
 
-        collectIteratorValueToArray(keys,this.data.keys());
+        collectIteratorValueToArray(keys, this.data.keys());
 
         const n = keys.length;
         for (let i = 0; i < n; i++) {

package/src/core/codegen/LineBuilder.js

@@ -16,6 +16,11 @@ class Line {
     }
 }
 
+const DEFAULT_INDENT_SPACES = 4;
+
+/**
+ * Useful for creating formatted snippets of code
+ */
 class LineBuilder {
 
     /**
@@ -32,7 +37,7 @@ class LineBuilder {
      *
      * @type {number}
      */
-    indentSpaces = 4;
+    indentSpaces = DEFAULT_INDENT_SPACES;
 
     /**
      *
@@ -79,8 +84,11 @@ class LineBuilder {
      * @returns {LineBuilder}
      */
     add(line_text) {
+
         const line = new Line(line_text, this.indentation);
+
         this.lines.push(line);
+
         return this;
     }
 
@@ -107,7 +115,7 @@ class LineBuilder {
     clear() {
         this.lines = [];
         this.indentation = 0;
-        this.indentSpaces = 4;
+        this.indentSpaces = DEFAULT_INDENT_SPACES;
     }
 
     /**
@@ -116,14 +124,16 @@ class LineBuilder {
      */
     build() {
         const result = [];
-        let i, j, l;
 
+        let i, j, l;
 
         const lines = this.lines;
+
         for (i = 0, l = lines.length; i < l; i++) {
             const line = lines[i];
 
             let indentString = '';
+
             for (j = 0; j < line.indentation * this.indentSpaces; j++) {
                 indentString += ' ';
             }
@@ -146,6 +156,8 @@ class LineBuilder {
 
         const n = lines.length;
 
+        // TODO detect indent
+
         for (let i = 0; i < n; i++) {
 
             r.add(lines[i]);

package/src/core/codegen/LineBuilder.spec.js

@@ -0,0 +1,7 @@
+import LineBuilder from "./LineBuilder.js";
+
+test("Empty builder should produce empty string", () => {
+    const b = new LineBuilder();
+
+    expect(b.build()).toEqual("");
+});