Bug 5299 – Protected inheritance is semantically undefined.

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P2
Component
dlang.org
Product
D
Version
D2
Platform
Other
OS
All
Creation time
2010-12-01T05:38:00Z
Last change time
2015-06-09T01:31:16Z
Keywords
spec
Assigned to
nobody
Creator
b.helyer
Blocks
2563

Comments

Comment #0 by b.helyer — 2010-12-01T05:38:20Z
http://www.digitalmars.com/d/2.0/class.html Inheritance is defined by the grammar to be allowed to take this form: Protection Identifier However, this is not mentioned anywhere, and the semantics are entirely undefined.
Comment #1 by issues.dlang — 2010-12-01T10:12:07Z
Well, that's exceptionally weird. It compiles too. D doesn't support any kind of inheritance but public. It seems like either it's a hold-over which was never removed (assuming that D supported private or protected inheritance in the past), or it was intended to make porting C++ code easier. But it seems to me that allowing and then ignoring (as the compiler presumably does) protection identifiers when inheriting classes is not good behavior. The fact that the compiler allows it though means that it's more than just a spec issue.
Comment #2 by smjg — 2011-07-24T09:38:50Z
Looking at (In reply to comment #1) > Well, that's exceptionally weird. It compiles too. D doesn't > support any kind of inheritance but public. It seems like either > it's a hold-over which was never removed (assuming that D supported > private or protected inheritance in the past), According to the description and first comment on issue 177, private/protected inheritance once worked. But it doesn't work now (1.069, 2.054). But it never made sense to have the feature in D. > or it was intended to make porting C++ code easier. D was never intended to be source-compatible with C++, or even C. Consequently, such a tiny concession makes no sense at all, and even less sense considering the fundamental differences between C++ classes and D classes. > But it seems to me that allowing and then ignoring (as the compiler > presumably does) protection identifiers when inheriting classes is > not good behavior. The fact that the compiler allows it though > means that it's more than just a spec issue. Issue 177 is "Remove inheritance protection". Essentially that has been done, but by ignoring attempts to use it rather than making it illegal. So that issue has been reduced to this one.
Comment #3 by smjg — 2011-07-24T09:44:36Z
*** Issue 177 has been marked as a duplicate of this issue. ***
Comment #4 by hoganmeier — 2011-09-15T13:32:28Z
Comment #5 by bugzilla — 2012-01-20T12:07:41Z
Comment #6 by yebblies — 2014-08-28T15:16:42Z
Pull to turn deprecation message into an error. https://github.com/D-Programming-Language/dmd/pull/3911
Comment #7 by github-bugzilla — 2014-08-31T05:23:06Z
Commits pushed to master at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/5ffc7d3784dc416769ff1a4aa0ab959ca42cd47e Fix Issue 5299 - Protected inheritance is semantically undefined Deprecation -> Error https://github.com/D-Programming-Language/dmd/commit/486e92da11fd567c32634a3085a31fa266634d77 Merge pull request #3911 from yebblies/issue5299 Issue 5299 - Turn base class protection into an error