From: Mat Martineau <mathew.j.martin...@linux.intel.com> Date: Thu, 21 Dec 2017 10:29:09 -0800
> Using a preprocessor directive to check for CONFIG_IPV6 in the middle of > a DECLARE_EVENT_CLASS macro's arg list causes sparse to report a series > of errors: > > ./include/trace/events/tcp.h:68:1: error: directive in argument list > ./include/trace/events/tcp.h:75:1: error: directive in argument list > ./include/trace/events/tcp.h:144:1: error: directive in argument list > ./include/trace/events/tcp.h:151:1: error: directive in argument list > ./include/trace/events/tcp.h:216:1: error: directive in argument list > ./include/trace/events/tcp.h:223:1: error: directive in argument list > ./include/trace/events/tcp.h:274:1: error: directive in argument list > ./include/trace/events/tcp.h:281:1: error: directive in argument list > > Once sparse finds an error, it stops printing warnings for the file it > is checking. This masks any sparse warnings that would normally be > reported for the core TCP code. > > Instead, handle the preprocessor conditionals in a couple of auxiliary > macros. This also has the benefit of reducing duplicate code. > > Cc: David Ahern <dsah...@gmail.com> > Signed-off-by: Mat Martineau <mathew.j.martin...@linux.intel.com> Applied.
