Thread overview
[Issue 3381] New: [tdpl] Incorrect assessment of overriding in triangular-shaped hierarchy
Oct 09, 2009
Gide Nwawudu
Oct 19, 2009
Don
Oct 19, 2009
Don
Oct 19, 2009
Leandro Lucarella
Nov 22, 2009
Bill Baxter
Nov 22, 2009
Leandro Lucarella
Dec 06, 2009
Walter Bright
October 09, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381

           Summary: [tdpl] Incorrect assessment of overriding in
                    triangular-shaped hierarchy
           Product: D
           Version: unspecified
          Platform: Other
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: DMD
        AssignedTo: nobody@puremagic.com
        ReportedBy: andrei@metalanguage.com


--- Comment #0 from Andrei Alexandrescu <andrei@metalanguage.com> 2009-10-08 20:36:16 PDT ---
This code doesn't compile:

interface VisualElement {
   void draw();
}

interface Actor {
}

interface VisualActor : Actor, VisualElement {
}

class Sprite3 : Actor, VisualActor {
    override void draw() { }
}

The error message is:

Error: function test.Sprite3.draw does not override any function

If I comment out the implementation of draw I get:

Error: class test.Sprite3 interface function VisualElement.draw isn't implemented

Of course, both of these can't be true at the same time :o). The original program should compile and run properly.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
October 09, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381


Gide Nwawudu <gide@nwawudu.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |diagnostic, rejects-valid
                 CC|                            |gide@nwawudu.com
         OS/Version|Linux                       |All


--- Comment #1 from Gide Nwawudu <gide@nwawudu.com> 2009-10-09 02:10:50 PDT ---
Looks like at dup of bug 3023.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
October 19, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381


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

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


--- Comment #2 from Don <clugdbug@yahoo.com.au> 2009-10-19 12:13:55 PDT ---
Ha! Compiler bugs are not usually this silly.
Applies to D1 as well. This also fixes bug 3023. I suspect it probably fixes
some other bugs too -- I'm amazed multiple inheritance has worked at all.
class.c, line 1198.

    // Skip if b has already appeared
    for (int k = 0; k < i; k++)
    {
-        if (b == interfaces[i])
+        if (b == interfaces[k])
        goto Lcontinue;
    }

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
October 19, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381


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

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


--- Comment #3 from Don <clugdbug@yahoo.com.au> 2009-10-19 12:15:37 PDT ---
*** Issue 3023 has been marked as a duplicate of this issue. ***

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
October 19, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381


Leandro Lucarella <llucax@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |llucax@gmail.com


--- Comment #4 from Leandro Lucarella <llucax@gmail.com> 2009-10-19 12:46:00 PDT ---
I wonder you (Andrei) don't search for existing bugs instead of blindly
creating new ones =(

If you want to mark all TDPL bugs by prefixing [tldp] to the subject, you can edit the old bug's title.

I'm saying this because I saw several bugs which were duplicates of old ones.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
October 19, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381



--- Comment #5 from Andrei Alexandrescu <andrei@metalanguage.com> 2009-10-19 12:55:57 PDT ---
(In reply to comment #4)
> I wonder you (Andrei) don't search for existing bugs instead of blindly
> creating new ones =(
> 
> If you want to mark all TDPL bugs by prefixing [tldp] to the subject, you can edit the old bug's title.
> 
> I'm saying this because I saw several bugs which were duplicates of old ones.

I'll try to look harder. My problem is that I never know what keywords to look for.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
November 22, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381


Bill Baxter <wbaxter@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |wbaxter@gmail.com


--- Comment #6 from Bill Baxter <wbaxter@gmail.com> 2009-11-22 06:53:43 PST ---
Looks like bug 2252 might be the same thing too.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
November 22, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381



--- Comment #7 from Leandro Lucarella <llucax@gmail.com> 2009-11-22 14:33:12 PST ---
SVN commit: http://www.dsource.org/projects/dmd/changeset/262

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 06, 2009
http://d.puremagic.com/issues/show_bug.cgi?id=3381


Walter Bright <bugzilla@digitalmars.com> changed:

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


--- Comment #8 from Walter Bright <bugzilla@digitalmars.com> 2009-12-06 00:46:29 PST ---
Fixed dmd 1.053 and 2.037

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