Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

New destroy implementation for value types and instinsic types #2124

Closed
wants to merge 1 commit into from
Closed

New destroy implementation for value types and instinsic types #2124

wants to merge 1 commit into from

Conversation

JinShil
Copy link
Contributor

@JinShil JinShil commented Mar 2, 2018

See discussion at #2115 for context.

This is my attempt to implement @andralex's bidding as he attempted to describe in the comments of #2115. Honestly, I don't understand his rationale and motivation for this implementation; it has a couple of special cases that seem arbitrary to me. It's also possible that I have not implemented this the way he envisioned.

But, I fear @andralex is losing his patience trying to describe what he wants and why, so I'm just going with it. Perhaps by having this implementation in front of us we can converge on understanding more quickly.

cc @schveiguy, @jmdavis

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @JinShil!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

@JinShil
Copy link
Contributor Author

JinShil commented Mar 2, 2018

Closing in favor of #2126

@JinShil JinShil closed this Mar 2, 2018
@JinShil JinShil deleted the new_destroy branch April 24, 2018 10:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants