http://d.puremagic.com/bugzilla/show_bug.cgi?id=61

           Summary: DStress test enum_39
           Product: D
           Version: 0.150
          Platform: All
               URL: http://dstress.kuehne.cn/nocompile/e/enum_39_A.d
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P3
         Component: DMD
        AssignedTo: bugzilla@digitalmars.com
        ReportedBy: unknown@simplemachines.org


The DStress tests enum_39_A through enum_39_E are all caused by one simple mistake in the parser.  It only results in incorrect error messages, but is also very easy to fix.

Attaching a patch to show the problem and solution.

Thanks,
-[Unknown]


-- 

http://d.puremagic.com/bugzilla/show_bug.cgi?id=61





------- Comment #1 from unknown@simplemachines.org  2006-03-22 02:01 -------
Created an attachment (id=7)
 --> (http://d.puremagic.com/bugzilla/attachment.cgi?id=7&action=view)
Pull the new loc while parsing EnumMembers.

This is pretty darn straight forward.

It's in the code that parses EnumMembers inside EnumDeclarations, and it sets loc to this->loc only once for the EnumDeclaration, but never for any of the EnumMembers (which may be on separate lines.)

This loc is only used to pass where the member was defined (so any error messages which depend on that member would have been wrong, including conflicts the other way around.)

-[Unknown]


-- 

http://d.puremagic.com/bugzilla/show_bug.cgi?id=61


larsivar@igesund.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |larsivar@igesund.net




------- Comment #2 from larsivar@igesund.net  2006-05-09 05:00 -------
This bug is listed as fixed in dmd 0.151, could someone verify it?


-- 

http://d.puremagic.com/bugzilla/show_bug.cgi?id=61


unknown@simplemachines.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




------- Comment #3 from unknown@simplemachines.org  2006-05-09 09:05 -------
DStress shows this as passing since 0.151:

http://dstress.kuehne.cn/www/dstress.html

-[Unknown]


--