Re: [PATCH] treewide: fix a bunch of typos (part 2)

From	Randy Dunlap <rdunlap@infradead.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] treewide: fix a bunch of typos (part 2)
Date	2016-08-08 16:40 +0200
Message-ID	<s3TSG-2yD-33@gated-at.bofh.it> (permalink)
References	<s3wMp-4w1-39@gated-at.bofh.it> <s3KPn-5bB-3@gated-at.bofh.it> <s3Lip-5AZ-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 08/07/16 22:20, Masahiro Yamada wrote:
> Hi Randy,
> 
> Thanks for taking a close look!
> 
> 
> 2016-08-08 13:50 GMT+09:00 Randy Dunlap <rdunlap@infradead.org>:
> 
>> --- a/net/sctp/transport.c
>> +++ b/net/sctp/transport.c
>> @@ -630,9 +630,7 @@ void sctp_transport_reset(struct sctp_transport *t)
>>         t->srtt = 0;
>>         t->rttvar = 0;
>>
>> -       /* Reset these additional varibles so that we have a clean
>> -        * slate.
>> -        */
>> +       /* Reset these additional variables so that we have a clean state. */
>>
>> * Nothing wrong with original comment.
> 
> 
> This hunk is doing three things:
> 
>  [1] varibles  -> variables
>  [2] slate   ->  state
>  [3] Make it into a single line
> 
> 
> Do you mean they are all unneeded changes?
> 

Sorry about that, I missed the varibles change. That change is correct.

The change of slate to state is not needed.

The single line change is OK also.

Thanks.


-- 
~Randy

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] treewide: fix a bunch of typos (part 2) Randy Dunlap <rdunlap@infradead.org> - 2016-08-08 07:00 +0200
  Re: [PATCH] treewide: fix a bunch of typos (part 2) Masahiro Yamada <yamada.masahiro@socionext.com> - 2016-08-08 07:30 +0200
    Re: [PATCH] treewide: fix a bunch of typos (part 2) Randy Dunlap <rdunlap@infradead.org> - 2016-08-08 16:40 +0200

csiph-web