ASTScope: Fix SubscriptDeclScope source range

slavapestov · slavapestov · commit 66873b94c9c4 · 2020-10-07T12:33:29.000-04:00
The source range needs to contain any attributes on the subscript
declaration, since those might create child scopes (@_specialize,
property wrappers, etc).
diff --git a/lib/AST/ASTScopeSourceRange.cpp b/lib/AST/ASTScopeSourceRange.cpp
@@ -226,7 +226,7 @@ SourceRange TopLevelCodeScope::getSourceRangeOfThisASTNode(
 
 SourceRange SubscriptDeclScope::getSourceRangeOfThisASTNode(
     const bool omitAssertions) const {
-  return decl->getSourceRange();
+  return decl->getSourceRangeIncludingAttrs();
 }
 
 SourceRange

Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,7 @@ SourceRange TopLevelCodeScope::getSourceRangeOfThisASTNode(`
`226`	`226`
`227`	`227`	`SourceRange SubscriptDeclScope::getSourceRangeOfThisASTNode(`
`228`	`228`	`const bool omitAssertions) const {`
`229`		`- return decl->getSourceRange();`
	`229`	`+ return decl->getSourceRangeIncludingAttrs();`
`230`	`230`	`}`
`231`	`231`
`232`	`232`	`SourceRange`