Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1475941

Re: [PATCH] checkpatch: Add a --strict test for macro argument reuse and precedence

From Joe Perches <joe@perches.com>
Newsgroups linux.kernel
Subject Re: [PATCH] checkpatch: Add a --strict test for macro argument reuse and precedence
Date 2016-09-04 16:50 +0200
Message-ID <sdGU9-71K-7@gated-at.bofh.it> (permalink)
References <sdpzY-1pL-13@gated-at.bofh.it> <sdrBM-2zP-19@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Sat, 2016-09-03 at 15:20 -0700, Joe Perches wrote:
> Add a test for reuse of macro arguments to highlight any possible
> side-effects from this reuse.
> 
> Avoid this check on token name pasting and when the
> argument is used in a typeof or a __builtin.
> 
> Add a test for macro arguents that have leading or trailing operators
> where the argument isn't parenthesized to avoid possible precedence
> issues.
> 
> These tests are noisy so make them --strict.

This should have been RFC.  Please do not apply this.

These tests are not just noisy, some false positives it
reports are just silly.

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Possible code defects: macros and precedence Joe Perches <joe@perches.com> - 2016-09-03 20:40 +0200
  Re: Possible code defects: macros and precedence Dan Carpenter <dan.carpenter@oracle.com> - 2016-09-03 22:20 +0200
    [PATCH] checkpatch: Add a --strict test for macro argument reuse and precedence Joe Perches <joe@perches.com> - 2016-09-04 00:30 +0200
      Re: [PATCH] checkpatch: Add a --strict test for macro argument  reuse and precedence Joe Perches <joe@perches.com> - 2016-09-04 16:50 +0200
  Re: Possible code defects: macros and precedence Julia Lawall <julia.lawall@lip6.fr> - 2016-09-04 12:20 +0200
    Re: Possible code defects: macros and precedence Joe Perches <joe@perches.com> - 2016-09-04 17:10 +0200

csiph-web