Date: Fri, 24 Aug 2007 12:47:43 -0400
From: Josef Sipek <jsipek@fsl.cs.sunysb.edu>
To: Mike Frysinger <vapier.adi@gmail.com>
Cc: SL Baur <steve@xemacs.org>, Andy Whitcroft <apw@shadowen.org>,
       Linux Kernel <linux-kernel@vger.kernel.org>,
       Michael Hennerich <michael.hennerich@analog.com>
Subject: Re: false positive in checkpatch.pl (complex macro values)
Message-ID: <20070824164741.GB1875@filer.fsl.cs.sunysb.edu>
References: <8bd0f97a0708240051w7fc00b01w8dc1b39fe606bfe4@mail.gmail.com> <46CEC3D4.40805@shadowen.org> <ed10ee420708240543r27db83e9l25300c2799c7c2cd@mail.gmail.com> <8bd0f97a0708240624v64b47b4l726a66f9766bc4a0@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <8bd0f97a0708240624v64b47b4l726a66f9766bc4a0@mail.gmail.com>
User-Agent: Mutt/1.5.16 (2007-07-16)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1101
Lines: 24

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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/