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

2021-07-23 Thread Alexey Utkin via Phabricator via cfe-commits
baramin added a comment. We have a regression after the fix: CPP-25899 CLion formatting does not match what clang-format produces if AlignConsecutiveAssignments: Consecutive Additional two spaces before `=` were added to indentation while form

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

2021-07-23 Thread Alexey Utkin via Phabricator via cfe-commits
baramin added a comment. You are right. The problem is in FromLegacyTimestamp(monitorFrequencyUsec), seconds(std::uint64_t(maxSampleAge)), maxKeepSamples)); line indentation. It is 6 instead of 4. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

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

2021-07-23 Thread Alexey Utkin via Phabricator via cfe-commits
baramin added a comment. This is true. .clang-format: Language:Cpp AlignConsecutiveAssignments: Consecutive BinPackArguments: false BinPackParameters: false ColumnLimit: 120 ConstructorInitializerIndentWidth: 4 ContinuationIndentWidth: 4 IndentWidth: 4 TabWidth:

[PATCH] D106773: [clang-format] Fix aligning with linebreaks #2

2021-07-26 Thread Alexey Utkin via Phabricator via cfe-commits
baramin added a comment. LGTM! It solves my problem. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106773/new/ https://reviews.llvm.org/D106773 ___ cfe-commits mailing list cfe-commits@lists.llvm