Skip to content

Conversation

@ilovepi
Copy link
Contributor

@ilovepi ilovepi commented Apr 30, 2025

No description provided.

Copy link
Contributor Author

ilovepi commented Apr 30, 2025

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

@llvmbot
Copy link
Member

llvmbot commented Apr 30, 2025

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

Author: Paul Kirth (ilovepi)

Changes

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

1 Files Affected:

  • (modified) clang-tools-extra/clang-doc/Representation.h (+1)
diff --git a/clang-tools-extra/clang-doc/Representation.h b/clang-tools-extra/clang-doc/Representation.h
index b0b34fc6f08dd..df5a7e69034a4 100644
--- a/clang-tools-extra/clang-doc/Representation.h
+++ b/clang-tools-extra/clang-doc/Representation.h
@@ -60,6 +60,7 @@ struct CommentInfo {
   // the vector.
   bool operator<(const CommentInfo &Other) const;
 
+  // TODO: The Kind field should be an enum, so we can switch on it easily
   SmallString<16>
       Kind; // Kind of comment (FullComment, ParagraphComment, TextComment,
             // InlineCommandComment, HTMLStartTagComment, HTMLEndTagComment,

Copy link
Contributor Author

ilovepi commented May 7, 2025

Merge activity

  • May 7, 7:22 PM EDT: A user started a stack merge that includes this pull request via Graphite.
  • May 7, 7:24 PM EDT: @ilovepi merged this pull request with Graphite.

@ilovepi ilovepi merged commit 83e1f34 into main May 7, 2025
6 of 9 checks passed
@ilovepi ilovepi deleted the users/ilovepi/clang-doc-comment branch May 7, 2025 23:24
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.

5 participants