jacob-abraham updated this revision to Diff 466386.
jacob-abraham added a comment.
fixed unit tests
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135422/new/
https://reviews.llvm.org/D135422
Files:
clang/lib/Format/UnwrappedLineFormatter.cpp
clang/unittests/Format/FormatTest.cpp
HazardyKnusperkeks added a comment.
The clang-format tests are (mainly) in `clang/unittests/Format/FormatTest.cpp`,
please put it there.
You should have a Format-Tests (or similar) build target.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135422/new/
https://reviews.llvm.org/D135422
jacob-abraham updated this revision to Diff 466166.
jacob-abraham added a comment.
Fix patch, sorry about that
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135422/new/
https://reviews.llvm.org/D135422
Files:
clang/lib/Format/UnwrappedLineFormatter.cpp
clang/test/Format/macro.cpp
jacob-abraham updated this revision to Diff 466165.
jacob-abraham added a comment.
Added regression test
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135422/new/
https://reviews.llvm.org/D135422
Files:
clang/test/Format/macro.cpp
Index: clang/test/Format/macro.cpp
HazardyKnusperkeks added a comment.
Please add a regression test.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135422/new/
https://reviews.llvm.org/D135422
___
cfe-commits mailing list
cfe-commits@lists
jacob-abraham created this revision.
jacob-abraham added reviewers: djasper, Typz.
jacob-abraham added a project: clang-format.
Herald added a project: All.
jacob-abraham requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This solves the issue
