Skip to content

Add in-place init goal#477

Merged
nikomatsakis merged 3 commits into
rust-lang:mainfrom
Darksonn:inplaceinit-2026
Jan 30, 2026
Merged

Add in-place init goal#477
nikomatsakis merged 3 commits into
rust-lang:mainfrom
Darksonn:inplaceinit-2026

Conversation

@Darksonn

@Darksonn Darksonn commented Jan 29, 2026

Copy link
Copy Markdown
Member

@BennoLossin BennoLossin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the overall feel of the goal, just a few things are a bit too ambiguous or brief, so I left two comments.

Comment thread src/2026/in-place-init.md
Comment on lines +49 to +51
| Task | Owner(s) | Notes |
| ----------- | -------- | ----- |
| Do the work | *owner* | |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like lowering the bar on this goal, since it's a very complex topic. We should provide a bit more detail in this table. How about:

  • collecting the different approaches in the wiki with the same format,
  • deciding on design criteria for a solution,
  • evaluating approaches against each other,

We could also include the idea of trying to get something minimal into a compiler experiment, see here.

We should also add the phased initialization for types not implementing Drop as a work item here.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also sign me up to write an RFC for placing functions. And unless someone else prefers to take the lead on that, please also feel free to assign the phased init RFC to me as well.

I think it might also be a good idea to give the rest of the items owners.

Comment thread src/2026/in-place-init.md Outdated
Comment thread src/2026/in-place-init.md Outdated
@Darksonn Darksonn requested a review from 1c3t3a January 30, 2026 14:36
Comment thread src/2026/in-place-init.md
@nikomatsakis nikomatsakis merged commit cbae12d into rust-lang:main Jan 30, 2026
2 of 3 checks passed
@Darksonn Darksonn deleted the inplaceinit-2026 branch January 31, 2026 09:30
@Darksonn Darksonn mentioned this pull request Jan 31, 2026
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants