#617 IdSetDefaultImplementation is buggy
Metadata
| Property | Value |
|---|---|
| Reported by | Carsten Gräser (graeser@math.fu-berlin.de) |
| Reported at | Oct 2, 2009 16:56 |
| Type | Bug Report |
| Version | Git (pre2.4) [autotools] |
| Operating System | Unspecified / All |
| Last edited by | Carsten Gräser (graeser@math.fu-berlin.de) |
| Last edited at | Nov 4, 2009 13:39 |
| Closed by | Carsten Gräser (graeser@math.fu-berlin.de) |
| Closed at | Nov 4, 2009 13:39 |
| Closed in version | Unknown |
| Resolution | Fixed |
| Comment | The remaining request is now covered by fs644. |
Description
IdSetDefaultImplementation used to provide a default implementation of the template subId(Entity,i) using id(). Since r5514 it pretends to be a default implemetation for subId(Entity,i,cc) using id().
However the code is incorrect. If we really want this we need a TMP since the type of subEntity(Entity,i) varies with the dynamic input cc.
Currently if MyGridIdSet derives from IdSetDefaultImplementation the default imp for subId(...) in IdSet uses the buggy subId() in IdSetDefaultImplementation even if it is (correctly) implemented in MyGridIdSet.
In ALU this hides the correct implementation of subId.