Skip to content

docs: clarify stale comment in cow1 owned_mutation test#2368

Open
DeoJin wants to merge 1 commit intorust-lang:mainfrom
DeoJin:fix/cow1-stale-comment
Open

docs: clarify stale comment in cow1 owned_mutation test#2368
DeoJin wants to merge 1 commit intorust-lang:mainfrom
DeoJin:fix/cow1-stale-comment

Conversation

@DeoJin
Copy link
Copy Markdown

@DeoJin DeoJin commented Mar 16, 2026

Summary

  • update the stale comment in exercises/19_smart_pointers/cow1.rs
  • clarify that o_mut() returns a mutable reference to already-owned data here
  • remove the misleading implication that �bs_all itself returns a reference

Closes #2303

Validation

  • unable to run cargo test owned_mutation --bin rustlings in this environment because Rust tooling (cargo,
    ustc) is not installed
  • verified the change is comment-only and scoped to the referenced exercise text

@senekor
Copy link
Copy Markdown
Contributor

senekor commented May 3, 2026

I don't see the improvement here. The two comments seem equally clear to me.

If an exercise file is changed, the solution file should be kept in sync.

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.

[19_smart_pointers/cow1.rs] Stale comment in owned_mutation test is confusing

2 participants