Opened 12 years ago

Closed 11 years ago

Last modified 6 years ago

#1908 closed defect (fixed)

Handle redeclares with multiple inheritance

Reported by: perost Owned by: perost
Priority: high Milestone: 1.9.4
Component: New Instantiation Version: trunk
Keywords: Cc:

Description

The new instantiation currently only applies redeclares to one baseclass when extending a class. This needs to be changed so that it applies redeclares to all applicable baseclasses. I think this only applies to element redeclares though.

Change History (5)

comment:1 follow-up: Changed 12 years ago by adrpo

This should be easy: just implement a new:

list<path> = SCodeLookup.lookupAllBaseClasses(name, inEnv, info);

that doesn't stop at the first extends but continues then use
this new function in SCodeFlattenRedeclare.replaceRedeclaredElementInEnv.
Did you fixed it already or should I do it?

Cheers,
Adrian Pop/

comment:2 in reply to: ↑ 1 Changed 12 years ago by perost

Replying to adrpo:

This should be easy

Not quite that easy, as always there's a bit more to consider (element redeclares and more). But I think it will be relatively easy, it's just a matter of extending the way it's currently done.

Did you fixed it already or should I do it?

It's the next thing on my todo list.

comment:3 Changed 11 years ago by perost

  • Resolution set to fixed
  • Status changed from new to closed

I think this has mostly been fixed, so I'm closing this ticket. If there are still problems we can open more specific tickets instead.

comment:4 Changed 8 years ago by dietmarw

  • Milestone changed from Future to pre1.9.4

It doesn't make sense to keep closed ticket in the "Future" milestone that were simply forgotten to assign to the correct milestone in the past.

comment:5 Changed 6 years ago by sjoelund.se

  • Milestone changed from pre1.9.4 to 1.9.4

Removing the pre1.9.4 milestone in favor of 1.9.4.

Note: See TracTickets for help on using tickets.