This repository has been archived on 2022-02-17. You can view files and clone it, but cannot push or open issues or pull requests.

Phil Carmody 497888cf69 treewide: fix potentially dangerous trailing ';' in #defined values/expressions ... All these are instances of #define NAME value; or #define NAME(params_opt) value; These of course fail to build when used in contexts like if(foo $OP NAME) while(bar $OP NAME) and may silently generate the wrong code in contexts such as foo = NAME + 1; /* foo = value; + 1; */ bar = NAME - 1; /* bar = value; - 1; */ baz = NAME & quux; /* baz = value; & quux; */ Reported on comp.lang.c, Message-ID: <ab0d55fe-25e5-482b-811e-c475aa6065c3@c29g2000yqd.googlegroups.com> Initial analysis of the dangers provided by Keith Thompson in that thread. There are many more instances of more complicated macros having unnecessary trailing semicolons, but this pile seems to be all of the cases of simple values suffering from the problem. (Thus things that are likely to be found in one of the contexts above, more complicated ones aren't.) Signed-off-by: Phil Carmody <ext-phil.2.carmody@nokia.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz>		2011-07-21 14:10:00 +02:00
..
Makefile	…
i810-i2c.c	drivers/video/i810/i810-i2c.c: fix i2c bus handling	2011-01-06 15:52:19 +09:00
i810.h	treewide: fix potentially dangerous trailing ';' in #defined values/expressions	2011-07-21 14:10:00 +02:00
i810_accel.c	Fix common misspellings	2011-03-31 11:26:23 -03:00
i810_dvt.c	i810fb: fix stack exploding	2009-12-16 07:20:02 -08:00
i810_gtf.c	…
i810_main.c	console: rename acquire/release_console_sem() to console_lock/unlock()	2011-01-26 10:50:06 +10:00
i810_main.h	…
i810_regs.h	…