[PATCH] D43180: [clang-format] Support text proto extensions

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 12 07:05:29 PST 2018


djasper added inline comments.


================
Comment at: unittests/Format/FormatTestTextProto.cpp:317
+
+TEST_F(FormatTestTextProto, FormatsExtensions) {
+  verifyFormat("[type] { key: value }");
----------------
It might be useful to attach a test case for what happens if the "[...]" does not fit on one line. I don't even know how I would format that, but it seems important to know and not accidentally modify the behavior.


Repository:
  rC Clang

https://reviews.llvm.org/D43180





More information about the cfe-commits mailing list