public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] rbtree: Fix typo in header comment
@ 2025-04-12  1:55 Po-Ying Chiu
  2025-04-14  8:43 ` David Sterba
  0 siblings, 1 reply; 3+ messages in thread
From: Po-Ying Chiu @ 2025-04-12  1:55 UTC (permalink / raw)
  To: peterz
  Cc: dsterba, wqu, mhiramat, beckerlee3, linux-kernel, jserv,
	Po-Ying Chiu

Correct "drammatically" to "dramatically" in the rbtree.h header comment.
This improves the readability of the header comment.

Signed-off-by: Po-Ying Chiu <charlie910417@gmail.com>
---
 include/linux/rbtree.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/rbtree.h b/include/linux/rbtree.h
index 8d2ba3749866..78a7a260c36f 100644
--- a/include/linux/rbtree.h
+++ b/include/linux/rbtree.h
@@ -7,7 +7,7 @@
   linux/include/linux/rbtree.h
 
   To use rbtrees you'll have to implement your own insert and search cores.
-  This will avoid us to use callbacks and to drop drammatically performances.
+  This will avoid us to use callbacks and to drop dramatically performances.
   I know it's not the cleaner way,  but in C (not in C++) to get
   performances and genericity...
 

base-commit: 2eb959eeecc64fa56e9f89a5fc496da297585cbe
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] rbtree: Fix typo in header comment
  2025-04-12  1:55 [PATCH] rbtree: Fix typo in header comment Po-Ying Chiu
@ 2025-04-14  8:43 ` David Sterba
  2025-04-15 12:49   ` [PATCH v2] " Po-Ying Chiu
  0 siblings, 1 reply; 3+ messages in thread
From: David Sterba @ 2025-04-14  8:43 UTC (permalink / raw)
  To: Po-Ying Chiu
  Cc: peterz, dsterba, wqu, mhiramat, beckerlee3, linux-kernel, jserv

On Sat, Apr 12, 2025 at 09:55:16AM +0800, Po-Ying Chiu wrote:
> Correct "drammatically" to "dramatically" in the rbtree.h header comment.
> This improves the readability of the header comment.
> 
> Signed-off-by: Po-Ying Chiu <charlie910417@gmail.com>
> ---
>  include/linux/rbtree.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/linux/rbtree.h b/include/linux/rbtree.h
> index 8d2ba3749866..78a7a260c36f 100644
> --- a/include/linux/rbtree.h
> +++ b/include/linux/rbtree.h
> @@ -7,7 +7,7 @@
>    linux/include/linux/rbtree.h
>  
>    To use rbtrees you'll have to implement your own insert and search cores.
> -  This will avoid us to use callbacks and to drop drammatically performances.
> +  This will avoid us to use callbacks and to drop dramatically performances.
>    I know it's not the cleaner way,  but in C (not in C++) to get
>    performances and genericity...

Maybe the paragraph can be rephrased entierely, "drop dramatically
performances" does not sound right to me.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH v2] rbtree: Fix typo in header comment
  2025-04-14  8:43 ` David Sterba
@ 2025-04-15 12:49   ` Po-Ying Chiu
  0 siblings, 0 replies; 3+ messages in thread
From: Po-Ying Chiu @ 2025-04-15 12:49 UTC (permalink / raw)
  To: dsterba
  Cc: beckerlee3, charlie910417, dsterba, jserv, linux-kernel, mhiramat,
	peterz, wqu

Correct "drammatically" to "dramatically" in the rbtree.h header comment.
This improves the readability of the header comment.

Signed-off-by: Po-Ying Chiu <charlie910417@gmail.com>
---
Changes since v1:
* Rephrased the whole paragraph to imporve readability.

 include/linux/rbtree.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/linux/rbtree.h b/include/linux/rbtree.h
index 8d2ba3749866..02b6733cce55 100644
--- a/include/linux/rbtree.h
+++ b/include/linux/rbtree.h
@@ -6,10 +6,10 @@
 
   linux/include/linux/rbtree.h
 
-  To use rbtrees you'll have to implement your own insert and search cores.
-  This will avoid us to use callbacks and to drop drammatically performances.
-  I know it's not the cleaner way,  but in C (not in C++) to get
-  performances and genericity...
+  To use rbtrees, you'll have to implement your own insert and search cores.
+  This avoids the need for callbacks, which would otherwise significantly reduce performance.
+  It may not be the cleanest approach, but in C (as opposed to C++), it is often
+  necessary to achieve both performance and genericity.
 
   See Documentation/core-api/rbtree.rst for documentation and samples.
 */
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-04-15 12:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-12  1:55 [PATCH] rbtree: Fix typo in header comment Po-Ying Chiu
2025-04-14  8:43 ` David Sterba
2025-04-15 12:49   ` [PATCH v2] " Po-Ying Chiu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox