Fix minor code tidyness issues (#123)

Fixes #115, #116
tableau · Oct 19, 2024 · 9b7e088 · 9b7e088
1 parent c52a47d
commit 9b7e088
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/query-graphs/src/hyper.ts b/query-graphs/src/hyper.ts
@@ -311,11 +311,10 @@ function colorRelativeExecutionTime(state: ConversionState) {
 
 // Sets the edge widths, relative to the number of output tuples
 function setEdgeWidths(state: ConversionState) {
-    let maxWidth = state.edgeWidths.reduce((p, v) => (p > v.width ? p : v.width), 0);
+    const maxWidth = state.edgeWidths.reduce((p, v) => (p > v.width ? p : v.width), 0);
     const minWidth = state.edgeWidths.reduce((p, v) => (p < v.width ? p : v.width), Infinity);
     if (minWidth == maxWidth) return;
     const factor = Math.max(maxWidth - minWidth, minWidth);
-    maxWidth = Math.max(maxWidth, 10);
     for (const edge of state.edgeWidths) {
         edge.node.edgeWidth = (edge.width - minWidth) / factor;
     }

diff --git a/query-graphs/src/xml.ts b/query-graphs/src/xml.ts
@@ -33,8 +33,10 @@ export function typesafeXMLParse(str: string): ParsedXML {
         mergeAttrs: false,
     });
     parser.parseString(str, (err: unknown, parsed: ParsedXML) => {
-        if (err) {
-            throw new Error("XML parse failed with '" + err + "'.");
+        if (err instanceof Error) {
+            throw new Error("XML parse failed with '" + err.message + "'.");
+        } else if (err) {
+            throw new Error("XML parse failed.");
         } else {
             result = parsed;
         }