This revision was automatically updated to reflect the committed changes.
Closed by commit rG66336056911e: [clang-format] Address fixme (authored by
HazardyKnusperkeks).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113320/new/
MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113320/new/
https://reviews.llvm.org/D113320
___
cfe-commits mailing list
HazardyKnusperkeks updated this revision to Diff 385248.
HazardyKnusperkeks marked an inline comment as done.
HazardyKnusperkeks added a reviewer: owenpan.
HazardyKnusperkeks added a comment.
Dropped braces.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
owenpan added a comment.
Looks okay.
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:3064-3068
+ for (const auto : Line.Tokens) {
+for (const auto : Node.Children)
+ printDebugInfo(ChildNode, "\nChild: ");
+
}
Nit: elide braces.
HazardyKnusperkeks created this revision.
HazardyKnusperkeks added reviewers: MyDeveloperDay, djasper.
HazardyKnusperkeks added a project: clang-format.
HazardyKnusperkeks requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Repository:
rG
