Mike
Mike
@WalterBright Is this PR still needed? The test case in https://issues.dlang.org/show_bug.cgi?id=17493 compiles fine, and https://issues.dlang.org/show_bug.cgi?id=17494 is marked RESOLVED.
> Probably still a good idea to add the test case to the test suite then. It looks like the test case was added in #6913
I don't see any reason to keep this open.
This appears to break code in druntime ``` timelimit -t 10 generated/linux/release/32/unittest/test_runner rt.minfo core.exception.InvalidMemoryOperationError@src/core/exception.d(702): Invalid memory operation ---------------- uncaught exception core.exception.InvalidMemoryOperationError@src/core/exception.d(702): Invalid memory operation ---------------- ```
I don't have any answers at this time, but there seems to be a fundamental design problem in the language if we have to resort to something like this. I'd...
That cyan looks pretty bad on white. Maybe a blue instead. Doesn't really matter to me, as long as it has more contrast.
This is better than my prior understanding of @andralex's specification as, at least, reference types and value types are treated uniformly. However, I still don't understand why having a destructor...
Breaking Higgs --------------- I removed one destructor from Higgs in an attempt to make it compatible with this PR, but there are still 3 more that need attention: https://github.com/higgsjs/Higgs/blob/dbe1fb993addfef98e900df6fe2e1e3afe8ecae4/source/runtime/object.d#L512 https://github.com/higgsjs/Higgs/blob/dbe1fb993addfef98e900df6fe2e1e3afe8ecae4/source/runtime/vm.d#L805...
> no attempts of fixing bugs regarding the class deconstructor when it comes to attributes? That's out of scope for now, and probably needs changes to be made in DMD.
> In what way are they broken? I believe it is their mere existence, causing the object to be re-initialized, that causes problems for the Higg's test suite. But, I...