Thread overview
[Issue 4278] New: undo limitations of bug3500's fix
Jun 05, 2010
Brad Roberts
[Issue 4278] allow inlining of super calls (undo limitations of bug3500's fix)
Aug 28, 2010
Walter Bright
Aug 29, 2010
Walter Bright
Aug 30, 2010
Haruki Shigemori
June 05, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4278

           Summary: undo limitations of bug3500's fix
           Product: D
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: NEW
          Keywords: patch
          Severity: normal
          Priority: P2
         Component: DMD
        AssignedTo: nobody@puremagic.com
        ReportedBy: braddr@puremagic.com
        Depends on: 3500


--- Comment #0 from Brad Roberts <braddr@puremagic.com> 2010-06-05 14:47:52 PDT ---
Created an attachment (id=655)
allow inlining of super.id(arg) calls

Bug 3500 reported bugginess with the inlining of super.id(args).  The fix there was to not inline that style call.  The patch attached to this report removes that block and fixes the underlying problem.

Now, the semantic pass rewrites super.id(args) to a direct call to the
baseclass.id(args).

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


Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |bugzilla@digitalmars.com
         Resolution|                            |FIXED


--- Comment #1 from Walter Bright <bugzilla@digitalmars.com> 2010-08-28 12:25:06 PDT ---
http://www.dsource.org/projects/dmd/changeset/640

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


Walter Bright <bugzilla@digitalmars.com> changed:

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


--- Comment #2 from Walter Bright <bugzilla@digitalmars.com> 2010-08-29 14:53:43 PDT ---
This patch produced a regression (from DFL):

---------
// main.d
import other;

class Foo : OtherModuleClass
{
        override void foo()
        {
                super.foo();
        }
}

void main()
{
        new Foo();
}

-----------
// other.d
class OtherModuleClass
{
        protected void foo()
        {
        }
}
-----------


> >dmd main other
main.d(8): Error: class other.OtherModuleClass member foo is not accessible

Reported by SHOO.

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


Haruki Shigemori <rayerd.wiz@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rayerd.wiz@gmail.com
         Depends on|3500                        |4728


--- Comment #3 from Haruki Shigemori <rayerd.wiz@gmail.com> 2010-08-29 17:07:10 PDT ---
Probably depends on 4728

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