http://bugzilla.kernel.org/show_bug.cgi?id=13519 --- Comment #1 from Matthew Wilcox <matthew@xxxxxx> 2009-06-12 12:32:31 --- On Fri, Jun 12, 2009 at 12:25:13PM +0000, bugzilla-daemon@xxxxxxxxxxxxxxxxxxx wrote: > i have detected a wrong macro call in file qla_sup.c. To detect this issue i > used the static code analysis tool cppcheck. It prints the following output: > > [linux-2.6.30/drivers/scsi/qla2xxx/qla_sup.c:352]: (error) Syntax error. Not > enough parameters for macro 'qla_printk'. > > Take a look at file qla_sup.c line 352 > > DEBUG9_10(qla_printk( > "NVRAM didn't go ready...\n")); > > > The maro definition is in file qla_def.h (2657): > > #define qla_printk(level, ha, format, arg...) \ > dev_printk(level , &((ha)->pdev->dev) , format , ## arg) > > As you can see, it needs more than one parameter. I believe your tool is defective. As I understand the way variadic macros work, if there is no 'arg', the ## operator swallows up the preceeding symbol (ie the ',') and so you'll get: dev_printk(level , &((ha)->pdev->dev) , format); which is perfectly fine. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching the assignee of the bug. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
