Skip to content

Conversation

@ilovepi
Copy link
Contributor

@ilovepi ilovepi commented Apr 19, 2025

No description provided.

@ilovepi ilovepi marked this pull request as ready for review April 19, 2025 01:04
Copy link
Contributor Author

ilovepi commented Apr 19, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@llvmbot
Copy link
Member

llvmbot commented Apr 19, 2025

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

Author: Paul Kirth (ilovepi)

Changes

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

1 Files Affected:

  • (modified) clang-tools-extra/clang-doc/Representation.cpp (+1-1)
diff --git a/clang-tools-extra/clang-doc/Representation.cpp b/clang-tools-extra/clang-doc/Representation.cpp index 0947ecff72272..54d2cb58ea2d9 100644 --- a/clang-tools-extra/clang-doc/Representation.cpp +++ b/clang-tools-extra/clang-doc/Representation.cpp @@ -215,7 +215,7 @@ void SymbolInfo::merge(SymbolInfo &&Other) { // Unconditionally extend the list of locations, since we want all of them. std::move(Other.Loc.begin(), Other.Loc.end(), std::back_inserter(Loc)); llvm::sort(Loc); - auto Last = std::unique(Loc.begin(), Loc.end()); + auto *Last = std::unique(Loc.begin(), Loc.end()); Loc.erase(Last, Loc.end()); mergeBase(std::move(Other)); } 
Copy link
Contributor Author

ilovepi commented Apr 19, 2025

Merge activity

  • Apr 19, 2:32 AM EDT: A user started a stack merge that includes this pull request via Graphite.
  • Apr 19, 2:34 AM EDT: A user merged this pull request with Graphite.
@ilovepi ilovepi merged commit e27aa12 into main Apr 19, 2025
15 checks passed
@ilovepi ilovepi deleted the users/ilovepi/clang-doc-autoptr branch April 19, 2025 06:34
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants