[clang-tools-extra] [clang-tidy] Clarify diagnostics of bugprone-sizeof-expression (PR #95550)

Donát Nagy via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 17 06:26:56 PDT 2024


NagyDonat wrote:

> Consider replacing ":" separator with ","

I switched to ":" (instead of the ";" that was used previously) to emphasize that the second part of the message is an explanation/reason for the first part (and not a second, separate issue). However I can switch to "," or ";" if that option turns out to be more popular.

@ rewiewers: Which symbol would you prefer in these messages?
(My preferences are dot > keep the semicolon ≳ comma.)

https://github.com/llvm/llvm-project/pull/95550


More information about the cfe-commits mailing list