[PATCH] D141547: Fix format for `case` in .proto files

2023-01-16 Thread Krasimir Georgiev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGcaf393da1823: Fix format for `case` in .proto files (authored by fowles, committed by krasimir). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141547/new/ h

[PATCH] D141547: Fix format for `case` in .proto files

2023-01-16 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir accepted this revision. krasimir added a comment. Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141547/new/ https://reviews.llvm.org/D141547 ___ cfe-commits mailing list cfe-commits@l

[PATCH] D141547: Fix format for `case` in .proto files

2023-01-13 Thread Matt Kulukundis via Phabricator via cfe-commits
fowles updated this revision to Diff 488979. fowles added a comment. Move from lit test to unit test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141547/new/ https://reviews.llvm.org/D141547 Files: clang/lib/Format/UnwrappedLineParser.cpp cla

[PATCH] D141547: Fix format for `case` in .proto files

2023-01-12 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir added inline comments. Comment at: clang/test/Format/case.proto:15 +repeated int32 fizz = 1; +} could you please turn this into an unit test here instead > https://github.com/llvm/llvm-project/blob/main/clang/unittests/Format/FormatTestProto.cpp. Rep

[PATCH] D141547: Fix format for `case` in .proto files

2023-01-11 Thread Eric Christopher via Phabricator via cfe-commits
echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141547/new/ https://reviews.llvm.org/D141547 __

[PATCH] D141547: Fix format for `case` in .proto files

2023-01-11 Thread Matt Kulukundis via Phabricator via cfe-commits
fowles created this revision. fowles added a reviewer: krasimir. Herald added a project: All. fowles requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fix format for `case` in .proto files Repository: rG LLVM Github Monorepo https://revi