[PATCH] D33440: clang-format: properly handle Q_UNUSED and QT_REQUIRE_VERSION

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 24 05:35:04 PDT 2017


djasper added a comment.

I don't. Only if they start out to be on the same line. As long as I start with:

  class C {
    void foo(int a, int b) {
      Q_UNUSED(a)
      Q_UNUSED(a)
      return b;
    }
  };

clang-format leaves this alone. That's good enough I think and we don't want to add more special handling for macro-DSLs than strictly necessary.


https://reviews.llvm.org/D33440





More information about the cfe-commits mailing list