Skip to content

docs: fix diskio comment wording#4829

Open
Rohan5commit wants to merge 1 commit intorust-lang:mainfrom
Rohan5commit:docs/diskio-comment-reused-wording-20260424
Open

docs: fix diskio comment wording#4829
Rohan5commit wants to merge 1 commit intorust-lang:mainfrom
Rohan5commit:docs/diskio-comment-reused-wording-20260424

Conversation

@Rohan5commit
Copy link
Copy Markdown

Summary

  • fix re-used in a source comment in diskio

Related issue

  • N/A (minor comment wording fix)

Guideline alignment

Validation

  • git diff --check

Copilot AI review requested due to automatic review settings April 24, 2026 03:18
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates a Rust doc comment in diskio to improve wording, without changing runtime behavior.

Changes:

  • Adjusted the hyphenation of “re-used” → “reused” in a FileBuffer method comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

View changes since this review

Comment thread src/diskio/mod.rs

impl FileBuffer {
/// All the buffers space to be re-used when the last reference to it is dropped.
/// All the buffers space to be reused when the last reference to it is dropped.
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

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

The doc comment reads a bit ungrammatically (“All the buffers space…”). Consider rephrasing to something like “All buffer space can be reused when the last reference is dropped” for clarity.

Suggested change
/// All the buffers space to be reused when the last reference to it is dropped.
/// All buffer space can be reused when the last reference is dropped.

Copilot uses AI. Check for mistakes.
@djc
Copy link
Copy Markdown
Contributor

djc commented Apr 24, 2026

Please address the Copilot feedback.

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