Skip to content

Conversation

@vitalybuka
Copy link
Collaborator

Fix container-overflow after 9189d84.

Created using spr 1.3.4
@llvmbot
Copy link
Member

llvmbot commented Mar 6, 2025

@llvm/pr-subscribers-clangd

@llvm/pr-subscribers-clang-tools-extra

Author: Vitaly Buka (vitalybuka)

Changes

Fix container-overflow after 9189d84.


Full diff: https://github.com/llvm/llvm-project/pull/130151.diff

1 Files Affected:

  • (modified) clang-tools-extra/clangd/refactor/tweaks/AnnotateHighlightings.cpp (+2-1)
diff --git a/clang-tools-extra/clangd/refactor/tweaks/AnnotateHighlightings.cpp b/clang-tools-extra/clangd/refactor/tweaks/AnnotateHighlightings.cpp
index 3a320260238b6..5432b1ef37ec2 100644
--- a/clang-tools-extra/clangd/refactor/tweaks/AnnotateHighlightings.cpp
+++ b/clang-tools-extra/clangd/refactor/tweaks/AnnotateHighlightings.cpp
@@ -51,7 +51,8 @@ Expected<Tweak::Effect> AnnotateHighlightings::apply(const Selection &Inputs) {
         *Inputs.AST, /*IncludeInactiveRegionTokens=*/true);
   } else {
     // Store the existing scopes.
-    const auto &BackupScopes = Inputs.AST->getASTContext().getTraversalScope();
+    const std::vector<Decl *> BackupScopes =
+        Inputs.AST->getASTContext().getTraversalScope();
     // Narrow the traversal scope to the selected node.
     Inputs.AST->getASTContext().setTraversalScope(
         {const_cast<Decl *>(CommonDecl)});

@vitalybuka vitalybuka closed this Mar 6, 2025
@vitalybuka
Copy link
Collaborator Author

9189d84 is reverted already

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants