Re: [PATCH 1/1] lib/test: Fix spelling mistakes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 7, 2021 at 11:56 AM Leizhen (ThunderTown)
<thunder.leizhen@xxxxxxxxxx> wrote:
> On 2021/6/7 16:52, Leizhen (ThunderTown) wrote:
> > On 2021/6/7 16:39, Andy Shevchenko wrote:
> >> On Mon, Jun 7, 2021 at 6:21 AM Zhen Lei <thunder.leizhen@xxxxxxxxxx> wrote:
> >>
> >>> Fix some spelling mistakes in comments:
> >>> thats ==> that's
> >>> unitialized ==> uninitialized
> >>> panicing ==> panicking
> >>> sucess ==> success
> >>> possitive ==> positive
> >>> intepreted ==> interpreted
> >>
> >> Thanks for the fix! Is it done with the help of the codespell tool? If
> >> not, can you run it and check if it suggests more fixes?
> >
> > Yes, it's detected by codespell tool. But to avoid too many changes in one patch, I tried
> > breaking it down into smaller patches(If it can be classified) to make it easier to review.
> > In fact, the other patch I just posted included the rest.
>
> https://lkml.org/lkml/2021/6/7/151
>
> All the remaining spelling mistakes are fixed by the patch above. I can combine the two of
> them into one patch if you think it's necessary.

No, it's good to keep them split. What I meant is to use the tool
against the same subset of the files you have done your patch for. But
please mention in the commit message that you have used that tool, so
reviewers will not waste time on the comments like mine.


-- 
With Best Regards,
Andy Shevchenko



[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux