[PATCH] D37973: [clang-format] Fix regression about short functions after #else

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 2 08:20:53 PDT 2017


sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

This test looks like it was intended to catch some case, maybe we're now mishandling some case like

  if (foo)
  {
  }
  else
  {
    // this can now be merged
  }

But there's no testcase and I'm guessing, so let's fix the `#else` bug :-)


https://reviews.llvm.org/D37973





More information about the cfe-commits mailing list