git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 3/5] Dangling commas in enum is not portable (resend)

From
GVGary V. Vaughan <git@mlists.thewrittenword.com>
Date
Apr 25, 2010, 08:35 UTC
Message-ID
<20100315031718.GA21130@thor.il.thewrittenword.com>
In-Reply-To
<94a0d4531003120118l2a5c69ebh7e62ea8630f94738@mail.gmail.com>
On Fri, Mar 12, 2010 at 11:18:47AM +0200, Felipe Contreras wrote:
Show 8 quoted lines
> On Thu, Mar 11, 2010 at 6:32 PM, Gary V. Vaughan
> <git@mlists.thewrittenword.com> wrote:
> > Unfortunately there are a still a lot of vendor compilers that choke
> > when the last entry in an enum has a dangling comma (irix is one, and
> > I can get you a list of which of our particular platforms exhibit this
> > problem).
> 
> I would appreciate that list if it's not much trouble.

I take it back. I've been porting this patch forward from our builds of previous releases, and at some point it certainly was necessary (otherwise I wouldn't have written the patch)... and yet, all of our platforms are now able to compile a minimal test program in C. Either the offending vendor compilers have issued an update to accept the dangling comma syntax, or we have retired the machines that used to have this problem since I originally wrote the patch.

It's notable that aCC (the HP C++ compiler) and CC (the SunOS C++ compiler) both issue a warning on dangling enum commas, but even they still compile successfully.

Sorry for the noise,
-- 
Gary V. Vaughan (gary@thewrittenword.com)
Previous: Felipe ContrerasNext: Tor Arntsen
Message 3 of 5 in “Dangling commas in enum is not portable (resend)”
  1. 3/5 Dangling commas in enum is not portable (resend)Gary V. Vaughan, Mar 11, 2010
  2. Felipe ContrerasMar 12, 2010
  3. Gary V. VaughanApr 25, 2010
  4. Tor ArntsenApr 25, 2010
  5. Gary V. VaughanApr 25, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.