diff options
author | Joe Perches <joe@perches.com> | 2011-07-26 02:13:22 +0200 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-07-26 05:57:16 +0200 |
commit | 7d2367af0b09f8028dc5c1b1919bb82d141c2afb (patch) | |
tree | e2645027cae1b716443e2659bda5364cb101d4e0 /scripts/gcc-x86_64-has-stack-protector.sh | |
parent | drivers/firmware/sigma.c needs MODULE_LICENSE (diff) | |
download | linux-7d2367af0b09f8028dc5c1b1919bb82d141c2afb.tar.xz linux-7d2367af0b09f8028dc5c1b1919bb82d141c2afb.zip |
checkpatch: suggest using min_t or max_t
A common issue with min() or max() is using a cast on one or both of the
arguments when using min_t/max_t could be better.
Add cast detection to uses of min/max and suggest an appropriate use of
min_t or max_t instead.
Caveat: This only works for min() or max() on a single line.
It does not find min() or max() split across multiple lines.
This does find:
min((u32)foo, bar);
But it does not find:
max((unsigned long)foo,
bar);
Suggested-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts/gcc-x86_64-has-stack-protector.sh')
0 files changed, 0 insertions, 0 deletions