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

From: Randy Dunlap
Date: Mon Aug 08 2016 - 10:38:57 EST


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@xxxxxxxxxxxxx>:
>
>> --- 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