[PATCH] D98214: [clang-format] Fix aligning with linebreaks

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 23 13:13:50 PDT 2021


HazardyKnusperkeks added a comment.

In D98214#2900803 <https://reviews.llvm.org/D98214#2900803>, @baramin wrote:

> You are right.
>
> The problem is in
>
>   FromLegacyTimestamp<milliseconds>(monitorFrequencyUsec), seconds(std::uint64_t(maxSampleAge)), maxKeepSamples));
>
> line indentation. It is 6 instead of 4.

What you call a regression I call a fix. :)
I can not see the .clang-format on you linked issue.

If we agree on that it is a bug, I will try to fix it, but that decision have others to do. (And then I will need the .clang-format.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98214



More information about the cfe-commits mailing list