Skip to content
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

In tstate::auxiliary::constraint, find a way to "freeze" the descs field #2539

Closed
catamorphism opened this issue Jun 7, 2012 · 1 comment
Closed

In tstate::auxiliary::constraint, find a way to "freeze" the descs field #2539

catamorphism opened this issue Jun 7, 2012 · 1 comment
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup.
Milestone
old 1.0 milestone

Comments

@catamorphism
Copy link
Contributor

This is a supreme yak-shave, but the type tstate::auxiliary::constraint has a mutable vector (dvec) field called descs, but it only needs to be mutable during the collect_locals part of typestate. Ideally collect_locals would use this type internally, then return a different type that has an immutable vector field. But I'm not sure if there's a good way to turn a dvec into an immutable vector.

Contingent on typestate not getting removed.
@ghost ghost assigned catamorphism Jun 7, 2012
@nikomatsakis
Copy link
Contributor

Not relevant as of commit 41a21f0, which removed current incarnation of typestate.

@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#2539 - oli-obk:rustup, r=oli-obk
beed5ed 
Rustup

for rust-lang#101649
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup.
Projects
None yet
Milestone
old 1.0 milestone
Development

No branches or pull requests
2 participants
@nikomatsakis @catamorphism