Skip to contentSkip to conversation

rust: rbtree: fix minor typos in comments

PATCH Superseded Atharv Dubey 2 participants 3 messages Dec 1, 2025 Rust

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 rust: rbtree: fix minor typos in comments

This patch corrects a few small grammatical errors in the documentation comments to improve clarity and readability.

---
 rust/kernel/rbtree.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
rust/kernel/rbtree.rs +2 -2
index b8fe6be6fcc4..6c9708201fd7 100644
@@ -732,7 +732,7 @@ pub struct Cursor<'a, K, V> {
 }
 
 // SAFETY: The [`Cursor`] has exclusive access to both `K` and `V`, so it is sufficient to require them to be `Send`.
-// The cursor only gives out immutable references to the keys, but since it has excusive access to those same
+// The cursor only gives out immutable references to the keys, but since it has exclusive access to those same
 // keys, `Send` is sufficient. `Sync` would be okay, but it is more restrictive to the user.
 unsafe impl<'a, K: Send, V: Send> Send for Cursor<'a, K, V> {}
 
@@ -980,7 +980,7 @@ pub struct IterMut<'a, K, V> {
 }
 
 // SAFETY: The [`IterMut`] has exclusive access to both `K` and `V`, so it is sufficient to require them to be `Send`.
-// The iterator only gives out immutable references to the keys, but since the iterator has excusive access to those same
+// The iterator only gives out immutable references to the keys, but since the iterator has exclusive access to those same
 // keys, `Send` is sufficient. `Sync` would be okay, but it is more restrictive to the user.
 unsafe impl<'a, K: Send, V: Send> Send for IterMut<'a, K, V> {}
 
Signature · 1 line
2.43.0
#1 lore raw
Quoted 2 lines · Atharv Dubey

This patch corrects a few small grammatical errors in the

s/few/couple?

It is also not a grammatical error, no? i.e. it is just a typo within a word.

Quoted 1 line

Thanks for doing these; however, since you already did other "good first issues", please do not do more to let others pick them up :)

It is why I typically write "...to use it as a test to submit your first patch to the kernel." in the issue text.

We can take this one since you did it already anyway. Please add the Suggested-by tag as well.

Thanks!

Cheers,
Miguel

#2 lore raw
Quoted 24 lines · Miguel Ojeda
Quoted 2 lines · Atharv Dubey

This patch corrects a few small grammatical errors in the

s/few/couple?

It is also not a grammatical error, no? i.e. it is just a typo within a word. Thanks for the feedback !, I'll update the commit message according to your suggestion.

Quoted 1 line

Thanks for doing these; however, since you already did other "good first issues", please do not do more to let others pick them up :)

It is why I typically write "...to use it as a test to submit your first patch to the kernel." in the issue text.

We can take this one since you did it already anyway. Please add the Suggested-by tag as well.
Got it, will keep this in mind for future contributions.
Thanks!

Cheers,
Miguel