Opened 12 years ago

Closed 12 years ago

Last modified 7 years ago

#1908 closed defect (fixed)

Handle redeclares with multiple inheritance

Reported by: Per Östlund Owned by: Per Östlund
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 by Adrian Pop, 12 years ago

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/

in reply to:  1 comment:2 by Per Östlund, 12 years ago

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 by Per Östlund, 12 years ago

Resolution: fixed
Status: newclosed

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 by Dietmar Winkler, 9 years ago

Milestone: Futurepre1.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 by Martin Sjölund, 7 years ago

Milestone: pre1.9.41.9.4

Removing the pre1.9.4 milestone in favor of 1.9.4.

Note: See TracTickets for help on using tickets.