Messages in this thread | | | From | "Tobin C. Harding" <> | Subject | [PATCH v3 0/4] checkpatch: warn for use of %px | Date | Tue, 27 Feb 2018 13:24:19 +1100 |
| |
Hi Andrew,
This is a resurrection of a patch set from last December. There was some confusion (on my behalf) as to how patches to checkpatch got into the mainline. Are you willing (and able) to take patches to checkpatch.pl?
Patch 1 through 3 are cleanup/refactoring patches.
Patch 3 makes checkpatch emit a warning for usage of specifier %px.
You may remember that the initial idea for this was from yourself, v1 requested permission to use 'Suggested-by' tag. I didn't get comment on that so v2 removed the tag. (I'm not totally across when one should add the 'Suggested-by' tag.)
thanks, Tobin.
v3: - rebase onto 4.16-rc3 - separate 'remove unused variable' into it's own patch
v2: - change new sub name stat_real() -> get_stat_real() - add new sub get_stat_here() - move the addition of new sub routines into separate patches - add 'Acked-by' tag for Joe - remove 'Suggested-by' tag for Andrew Morton
Tobin C. Harding (4): checkpatch: add sub routine get_stat_real() checkpatch: remove unused variable declarations checkpatch: add sub routine get_stat_here() checkpatch: warn for use of %px
scripts/checkpatch.pl | 113 ++++++++++++++++++++++++++------------------------ 1 file changed, 58 insertions(+), 55 deletions(-)
-- 2.7.4
| |