Skip to content

Classes: implement destructors #2295

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
catamorphism opened this issue Apr 25, 2012 · 2 comments
Closed

Classes: implement destructors #2295

catamorphism opened this issue Apr 25, 2012 · 2 comments
Labels
A-codegen Area: Code generation A-type-system Area: Type system C-enhancement Category: An issue proposing an enhancement or a PR with one.
Milestone
0.3

Comments

@catamorphism
Copy link
Contributor

Implement dtors for classes, then get rid of resources (rewriting resource-using code to use classes with dtors instead).
@ghost ghost assigned catamorphism Apr 25, 2012
catamorphism added a commit that referenced this issue May 14, 2012
@catamorphism
First cut at dtors for classes
5428a22 
Classes with dtors should compile now. Haven't yet tested
whether they actually run correctly.

Beginnings of support for #2295, though that won't be done until
there's more test cases and resources are removed.
@catamorphism
Copy link
Contributor Author

Status: destructors are implemented, but can't be called correctly from the cycle collector (resource-cycle fails when ported to use classes). I think this is probably easy to fix for someone who understands the RTS. Not all test cases have been ported yet. I'm deferring this for the moment to work on more urgent bugs. If someone else worked on it, I wouldn't object.

@catamorphism
Copy link
Contributor Author

I'll just consider this done except for the cycle collector part, and file more specific bugs.

@catamorphism catamorphism removed their assignment Jun 16, 2014
bors added a commit to rust-lang-ci/rust that referenced this issue Sep 22, 2022
@bors
Auto merge of rust-lang#2295 - RalfJung:better-alloc-tracking, r=Ralf…
f9e0cf4 
…Jung

allocation tracking: also print size, alignment, kind of the allocation
celinval pushed a commit to celinval/rust-dev that referenced this issue Jun 4, 2024
@qinheping
Fix link to assess-scan-on-repos.sh in document (rust-lang#2295)
be4e578
jieyouxu added a commit to jieyouxu/rust that referenced this issue Apr 2, 2025
@jieyouxu
Merge pull request rust-lang#2295 from lolbinarycat/rustdoc-htmldocck
594eed9 
add new section on the `rustdoc` test suite
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-codegen Area: Code generation A-type-system Area: Type system C-enhancement Category: An issue proposing an enhancement or a PR with one.
Projects
None yet
Milestone
0.3
Development

No branches or pull requests
1 participant
@catamorphism