[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <871piuasb5.fsf@trenco.lwn.net>
Date: Mon, 09 Feb 2026 07:46:06 -0700
From: Jonathan Corbet <corbet@....net>
To: Min-Hsun Chang <chmh0624@...il.com>, visitorckw@...il.com
Cc: linux-kernel@...r.kernel.org, linux-doc@...r.kernel.org, Min-Hsun Chang
<chmh0624@...il.com>
Subject: Re: [PATCH] lib/min_heap: fix spelling and grammatical errors
Min-Hsun Chang <chmh0624@...il.com> writes:
> The Min Heap API documentation and header file contain a few minor
> spelling and grammatical mistakes.
>
> Fix the following:
> - Add a missing period at the end of a sentence in min_heap.rst.
> - Change "customise" to "customize" in min_heap.h for consistency
> with the documentation.
> - Change "on to" to "onto" in the description of min_heap_push.
>
> These changes improve the readability of the API and documentation.
>
> Signed-off-by: Min-Hsun Chang <chmh0624@...il.com>
> ---
> Documentation/core-api/min_heap.rst | 2 +-
> include/linux/min_heap.h | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/core-api/min_heap.rst b/Documentation/core-api/min_heap.rst
> index 9f57766581df..62439be2cdf6 100644
> --- a/Documentation/core-api/min_heap.rst
> +++ b/Documentation/core-api/min_heap.rst
> @@ -75,7 +75,7 @@ elements in the heap and swapping them. It contains two function pointers:
>
> - **less** is the comparison function used to establish the order of elements.
> - **swp** is a function for swapping elements in the heap. If swp is set to
> - NULL, the default swap function will be used, which swaps the elements based on their size
> + NULL, the default swap function will be used, which swaps the elements based on their size.
>
> Macro Wrappers
> ==============
> diff --git a/include/linux/min_heap.h b/include/linux/min_heap.h
> index 79ddc0adbf2b..cb1cc0c0e8bd 100644
> --- a/include/linux/min_heap.h
> +++ b/include/linux/min_heap.h
> @@ -40,7 +40,7 @@ typedef DEFINE_MIN_HEAP(char, min_heap_char) min_heap_char;
> #define __minheap_obj_size(_heap) sizeof((_heap)->data[0])
>
> /**
> - * struct min_heap_callbacks - Data/functions to customise the min_heap.
> + * struct min_heap_callbacks - Data/functions to customize the min_heap.
Both British and American spellings are fine in the kernel and do not
need to be fixed.
Thanks,
jon
Powered by blists - more mailing lists