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 <[email protected]>: > >> --- 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
