Clazy: improved explanations
ClosedPublic

Authored by winterz on Oct 14 2018, 12:46 PM.

Details

Summary

This patch does 2 things:

  1. also looks for explanations of manuallevel checkers
  2. provides a nice message if the user mistyped the checker name
Test Plan

$ clazy --explain tr-non-ladsfa
There is no explanation available for checker "tr-non-ladsfa"
Run 'clazy --explain' to see the list of all available checkers.

$ clazy --explain tr-non-

Explanation for checker tr-non-literal (manuallevel)

tr-non-literal

Finds calls to QObject::tr() with non literal argument.

Example: tr(myStr.toUtf8());

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
winterz requested review of this revision.Oct 14 2018, 12:46 PM
winterz created this revision.
smartins accepted this revision.Oct 25 2018, 8:54 PM

looks good!

This revision is now accepted and ready to land.Oct 25 2018, 8:54 PM
winterz closed this revision.Oct 25 2018, 9:06 PM