diff options
author | Joe Perches <joe@perches.com> | 2011-07-26 00:13:22 (GMT) |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-07-26 03:57:16 (GMT) |
commit | 7d2367af0b09f8028dc5c1b1919bb82d141c2afb (patch) | |
tree | e2645027cae1b716443e2659bda5364cb101d4e0 /crypto | |
parent | 27c46a2546c75c6814562e85b751e3d64c188ad5 (diff) | |
download | linux-fsl-qoriq-7d2367af0b09f8028dc5c1b1919bb82d141c2afb.tar.xz |
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 'crypto')
0 files changed, 0 insertions, 0 deletions