summaryrefslogtreecommitdiffstats
path: root/crypto/cast5_generic.c
diff options
context:
space:
mode:
authorJoe Perches <joe@perches.com>2014-04-03 14:49:21 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2014-04-03 16:21:14 -0700
commit5b9553abfc97f923b99868a04c3b3d99a6014163 (patch)
tree3d54a69799ee4e749d001d33834907b9e76460ae /crypto/cast5_generic.c
parent85ad978c626a425c0bacd137bf15403e867c9134 (diff)
downloadlinux-5b9553abfc97f923b99868a04c3b3d99a6014163.tar.gz
linux-5b9553abfc97f923b99868a04c3b3d99a6014163.tar.bz2
linux-5b9553abfc97f923b99868a04c3b3d99a6014163.zip
checkpatch: make "return is not a function" test quieter
This test is a bit noisy and opinions seem to agree that it should not warn in a lot more situations. It seems people agree that: return (foo || bar); and return foo || bar; are both acceptable style and checkpatch should be silent about them. For now, it warns on parentheses around a simple constant or a single function or a ternary. return (foo); return (foo(bar)); return (foo ? bar : baz); The last ternary test may be quieted in the future. Modify the deparenthesize function to only strip balanced leading and trailing parentheses. Signed-off-by: Joe Perches <joe@perches.com> Cc: Julia Lawall <julia.lawall@lip6.fr> Reviewed-by: Josh Triplett <josh@joshtriplett.org> Cc: Monam Agarwal <monamagarwal123@gmail.com> Cc: Greg KH <gregkh@linuxfoundation.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'crypto/cast5_generic.c')
0 files changed, 0 insertions, 0 deletions