lkml.org 
[lkml]   [2022]   [Oct]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] net/mptcp: fix repeated words in comments
On Sun, 23 Oct 2022, Bagas Sanjaya wrote:

> On Sat, Oct 22, 2022 at 03:05:27PM +0800, wangjianli wrote:
>> Delete the redundant word 'the'.
>>
>> Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
>> ---
>> net/mptcp/token.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/net/mptcp/token.c b/net/mptcp/token.c
>> index f52ee7b26aed..b817c2564300 100644
>> --- a/net/mptcp/token.c
>> +++ b/net/mptcp/token.c
>> @@ -287,7 +287,7 @@ EXPORT_SYMBOL_GPL(mptcp_token_get_sock);
>> * This function returns the first mptcp connection structure found inside the
>> * token container starting from the specified position, or NULL.
>> *
>> - * On successful iteration, the iterator is move to the next position and the
>> + * On successful iteration, the iterator is move to the next position and
>> * the acquires a reference to the returned socket.
>> */
>> struct mptcp_sock *mptcp_token_iter_next(const struct net *net, long *s_slot,
>
> NAK!
>
> Instead, slightly reword the comment above as "On successful iteration,
> the iterator moves to the next position and acquires a reference to the
> returned socket.".
>

Agree on this rewording.

This particular duplicated word came up before, and I thought it would be
best if the author sent a v2 - but they never did. I will fix this in the
MPTCP tree next week if there's no suitable v2 by then.

--
Mat Martineau
Intel

\
 
 \ /
  Last update: 2022-10-25 01:49    [W:0.040 / U:0.452 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site