Thread overview
[Issue 4134] New: Trailing comma allowed in parameter lists
Apr 29, 2010
Aziz Köksal
Apr 29, 2010
Ellery Newcomer
Apr 30, 2010
Simen Kjaeraas
Apr 30, 2010
Aziz Köksal
Apr 30, 2010
Don
April 29, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4134

           Summary: Trailing comma allowed in parameter lists
           Product: D
           Version: 1.057
          Platform: Other
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: DMD
        AssignedTo: nobody@puremagic.com
        ReportedBy: aziz.koeksal@gmail.com


--- Comment #0 from Aziz Köksal <aziz.koeksal@gmail.com> 2010-04-29 09:32:56 PDT ---
Example:

void main(char[][] args,)
{}

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
April 29, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4134


Ellery Newcomer <ellery-newcomer@utulsa.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ellery-newcomer@utulsa.edu


--- Comment #1 from Ellery Newcomer <ellery-newcomer@utulsa.edu> 2010-04-29 11:20:25 PDT ---
Sweet. How long has this feature been available?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
April 30, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4134


Simen Kjaeraas <simen.kjaras@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |simen.kjaras@gmail.com
         Resolution|                            |WONTFIX


--- Comment #2 from Simen Kjaeraas <simen.kjaras@gmail.com> 2010-04-30 03:43:48 PDT ---
This is allowed for simplicity with generated code, I believe. Makes it a bit easier to generate parameter lists, enumeration enums (darn those other enums), etc.

It's been around for as long as I can remember, and hopefully won't ever be 'fixed'.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
April 30, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4134



--- Comment #3 from Aziz Köksal <aziz.koeksal@gmail.com> 2010-04-30 05:50:36 PDT ---
(In reply to comment #2)
> This is allowed for simplicity with generated code, I believe. Makes it a bit easier to generate parameter lists, enumeration enums (darn those other enums), etc.
>

This would make more sense for array literals, because generating them should happen much more often than generating parameter lists.

I have no problem with trailing commas, but it's not consistently allowed everywhere.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
April 30, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4134


Don <clugdbug@yahoo.com.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |clugdbug@yahoo.com.au


--- Comment #4 from Don <clugdbug@yahoo.com.au> 2010-04-30 05:55:50 PDT ---
(In reply to comment #3)
> (In reply to comment #2)
> > This is allowed for simplicity with generated code, I believe. Makes it a bit easier to generate parameter lists, enumeration enums (darn those other enums), etc.
> >
> 
> This would make more sense for array literals, because generating them should happen much more often than generating parameter lists.
> 
> I have no problem with trailing commas, but it's not consistently allowed everywhere.

See bug 2477.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------