Re: false positive in checkpatch.pl (complex macro values)

From: Josef Sipek
Date: Fri Aug 24 2007 - 12:48:02 EST


On Fri, Aug 24, 2007 at 09:24:17AM -0400, Mike Frysinger wrote:
...
> $ grep AUREON_DEVICE_DESC ./sound/pci/ice1712/aureon.h -A 5
> #define AUREON_DEVICE_DESC "{Terratec,Aureon 5.1 Sky},"\
> "{Terratec,Aureon 7.1 Space},"\
> "{Terratec,Aureon 7.1 Universe}," \
> "{AudioTrak,Prodigy 7.1}," \
> "{AudioTrak,Prodigy 7.1 LT},"\
> "{AudioTrak,Prodigy 7.1 XT},"

This is a different thing. This is a long string, not a list of elements.
IMO, this one shouldn't have given a warning regardless of whether or not
the original case is valid.

Josef 'Jeff' Sipek.

--
If I have trouble installing Linux, something is wrong. Very wrong.
- Linus Torvalds
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/