[PATCH] D59627: [clang-format] Keep protobuf "package" statement on one line

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 25 08:45:59 PDT 2019


hokein added a comment.

This patch has caused a regression issue, see the test:

  verifyFormat("// Detached comment\n\n"
                   "// Leading comment\n"
                   "syntax = \"proto2\"; // trailing comment\n\n"
                   "// in foo.bar package\n"
                   "package foo.bar; // foo.bar package\n");

I have reverted it in rL356912 <https://reviews.llvm.org/rL356912>.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59627/new/

https://reviews.llvm.org/D59627





More information about the cfe-commits mailing list