Cleanup fixing som lingering issues

mermaid-js · Sep 14, 2022 · 9acdc0b · 9acdc0b
1 parent ac788a7
commit 9acdc0b
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 6 deletions.
diff --git a/src/diagrams/c4/c4Renderer.js b/src/diagrams/c4/c4Renderer.js
@@ -616,7 +616,6 @@ export const draw = function (_text, id, _version, diagObj) {
   globalBoundaryMaxY = conf.diagramMarginY;
 
   const title = diagObj.db.getTitle();
-  const c4type = diagObj.db.getC4Type(); // TODO: @knsv: remove this?
   let currentBoundarys = diagObj.db.getBoundarys('');
   // switch (c4type) {
   //   case 'C4Context':

diff --git a/src/diagrams/class/classRenderer.js b/src/diagrams/class/classRenderer.js
@@ -157,8 +157,6 @@ export const draw = function (text, id, _version, diagObj) {
     securityLevel === 'sandbox'
       ? select(sandboxElement.nodes()[0].contentDocument.body)
       : select('body');
-  // TODO: @knsv doc is not used, bug?
-  const doc = securityLevel === 'sandbox' ? sandboxElement.nodes()[0].contentDocument : document;
 
   // Fetch the default direction, use TD if none was found
   const diagram = root.select(`[id='${id}']`);

diff --git a/src/diagrams/mindmap/mindmapRenderer.js b/src/diagrams/mindmap/mindmapRenderer.js
@@ -192,9 +192,7 @@ function layoutMindmap(node, conf) {
   });
 
   // Merge the trees into a single tree
-  const result = mergeTrees(node, trees);
-  // TODO: @knsv The function is not called bug?
-  eachNode;
+  mergeTrees(node, trees);
   return node;
 }
 /**