[PATCH] D109210: [clang-tidy] Attach fixit to warning, not note, in add_new_check.py example

Whisperity via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 3 03:38:54 PDT 2021


whisperity added a comment.

Uuuh.. I get the sentiment, but this change breaks the very essence of the joke that the default generated code wants to pass on. It also does not showcase that we can emit notes, not just warnings.

How about `function %0 is insufficiently awesome, mark it as 'awesome_'!` in the warning text? (Note how the fix and the note's text diverged already.)

And so we should come up with a joke for the Note tag.

BTW, the test wasn't checking for the note's message at all?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109210



More information about the cfe-commits mailing list