Switched from Map to Set in code flow routine (minor cleanup).

microsoft · Sep 15, 2021 · f39520a · f39520a
1 parent a53e8af
commit f39520a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/pyright-internal/src/analyzer/typeEvaluator.ts b/packages/pyright-internal/src/analyzer/typeEvaluator.ts
@@ -18229,7 +18229,7 @@ export function createTypeEvaluator(
     // is specified, it returns true only if at least one control flow
     // path passes through sourceFlowNode.
     function isFlowNodeReachable(flowNode: FlowNode, sourceFlowNode?: FlowNode): boolean {
-        const visitedFlowNodeMap = new Map<number, true>();
+        const visitedFlowNodeMap = new Set<number>();
 
         function isFlowNodeReachableRecursive(flowNode: FlowNode, sourceFlowNode: FlowNode | undefined): boolean {
             let curFlowNode = flowNode;
@@ -18242,7 +18242,7 @@ export function createTypeEvaluator(
                 }
 
                 // Note that we've been here before.
-                visitedFlowNodeMap.set(curFlowNode.id, true);
+                visitedFlowNodeMap.add(curFlowNode.id);
 
                 if (curFlowNode.flags & FlowFlags.Unreachable) {
                     return false;