From d5290a81234483b706be8ce9651aa01e4be0ba57 Mon Sep 17 00:00:00 2001 From: Stella Huang Date: Wed, 15 Apr 2026 14:44:05 -0700 Subject: [PATCH] fix style --- .../pyright-internal/src/analyzer/binder.ts | 36 +++++++++---------- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/packages/pyright-internal/src/analyzer/binder.ts b/packages/pyright-internal/src/analyzer/binder.ts index f6eef44daa15..6def41b71463 100644 --- a/packages/pyright-internal/src/analyzer/binder.ts +++ b/packages/pyright-internal/src/analyzer/binder.ts @@ -1209,24 +1209,6 @@ export class Binder extends ParseTreeWalker { return false; } - // Helper method to determine if an expression is a non-empty list or tuple literal. - // This is a syntactic check, not a semantic one, so it's very fast. - // Guards against starred expressions ([*empty_list]) and comprehensions ([v for v in []]). - private _isNonEmptyListOrTupleLiteral(expr: ExpressionNode): boolean { - if (expr.nodeType === ParseNodeType.List) { - return ( - expr.d.items.length > 0 && - expr.d.items.every( - (item) => item.nodeType !== ParseNodeType.Unpack && item.nodeType !== ParseNodeType.Comprehension - ) - ); - } - if (expr.nodeType === ParseNodeType.Tuple) { - return expr.d.items.length > 0 && expr.d.items.every((item) => item.nodeType !== ParseNodeType.Unpack); - } - return false; - } - override visitFor(node: ForNode) { this._bindPossibleTupleNamedTarget(node.d.targetExpr); this._addInferredTypeAssignmentForVariable(node.d.targetExpr, node); @@ -2502,6 +2484,24 @@ export class Binder extends ParseTreeWalker { return true; } + // Helper method to determine if an expression is a non-empty list or tuple literal. + // This is a syntactic check, not a semantic one, so it's very fast. + // Guards against starred expressions ([*empty_list]) and comprehensions ([v for v in []]). + private _isNonEmptyListOrTupleLiteral(expr: ExpressionNode): boolean { + if (expr.nodeType === ParseNodeType.List) { + return ( + expr.d.items.length > 0 && + expr.d.items.every( + (item) => item.nodeType !== ParseNodeType.Unpack && item.nodeType !== ParseNodeType.Comprehension + ) + ); + } + if (expr.nodeType === ParseNodeType.Tuple) { + return expr.d.items.length > 0 && expr.d.items.every((item) => item.nodeType !== ParseNodeType.Unpack); + } + return false; + } + private _addTypingImportAliasesFromBuiltinsScope() { if (!this._fileInfo.builtinsScope) { return;