[PATCH] D135871: [clang-format][NFC] Handle language specific stuff at the top...

2022-10-24 Thread Björn Schäpers via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. HazardyKnusperkeks marked 2 inline comments as done. Closed by commit rG006bf8d81770: [clang-format][NFC] Handle language specific stuff at the top... (authored by

[PATCH] D135871: [clang-format][NFC] Handle language specific stuff at the top...

2022-10-16 Thread Owen Pan via Phabricator via cfe-commits
owenpan accepted this revision. owenpan added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/lib/Format/TokenAnnotator.cpp:3121 + return 1; +if (Right.is(tok::period)) + return 500; HazardyKnusperkeks wrote: >

[PATCH] D135871: [clang-format][NFC] Handle language specific stuff at the top...

2022-10-15 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added inline comments. Comment at: clang/lib/Format/TokenAnnotator.cpp:3121 + return 1; +if (Right.is(tok::period)) + return 500; owenpan wrote: > Otherwise, it wouldn't be NFC. It was Left == :: || Right == . Repository: rG

[PATCH] D135871: [clang-format][NFC] Handle language specific stuff at the top...

2022-10-14 Thread Owen Pan via Phabricator via cfe-commits
owenpan added inline comments. Comment at: clang/lib/Format/TokenAnnotator.cpp:3121 + return 1; +if (Right.is(tok::period)) + return 500; Otherwise, it wouldn't be NFC. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D135871: [clang-format][NFC] Handle language specific stuff at the top...

2022-10-13 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks created this revision. HazardyKnusperkeks added reviewers: owenpan, MyDeveloperDay, curdeius, rymiel. HazardyKnusperkeks added a project: clang-format. Herald added a project: All. HazardyKnusperkeks requested review of this revision. Herald added a project: clang. Herald added