MyDeveloperDay added a comment. In D98214#2900371 <https://reviews.llvm.org/D98214#2900371>, @baramin wrote:
> We have a regression after the fix: > CPP-25899 <https://youtrack.jetbrains.com/issue/CPP-25899> CLion formatting > does not match what clang-format produces > > if > > AlignConsecutiveAssignments: Consecutive > > Additional two spaces before `=` were added to indentation while formatting > > void SomeFunc() > { > using DcgmNs::Timelib::FromLegacyTimestamp; > using DcgmNs::Timelib::ToLegacyTimestamp; > using DcgmNs::Utils::GetMaxAge; > using namespace std::chrono; > newWatcher.maxAgeUsec = ToLegacyTimestamp(GetMaxAge( > FromLegacyTimestamp<milliseconds>(monitorFrequencyUsec), > seconds(std::uint64_t(maxSampleAge)), maxKeepSamples)); > newWatcher.isSubscribed = subscribeForUpdates ? 1 : 0; > } At first glance this looks correct to me, `newWatcher.maxAgeUsec =` and `newWatcher.isSubscribed =` are considered consecutive. Am I missing something? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98214/new/ https://reviews.llvm.org/D98214 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
