[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block |
Date: |
Thu, 13 Dec 2018 18:01:29 +0000 |
On Thu, 13 Dec 2018 at 17:57, Wainer dos Santos Moschetta
<address@hidden> wrote:
>
> Eduardo Habkost pointed out a malformed block of comments on my
> patch [1] that I had ran checkpatch.pl and no warn/error was
> reported. Then I realized the script does not catch such as
> case (or it had a bug).
>
> It turns out that checkpatch.pl does not parse comment blocks (If I understood
> its code correctly...). So I implemented a checker that warns about:
> 1. block doesn't begin on its own line.
> Example:
> /* blah blah
> * and blah blah
> */
I sent a patch to do this a little while back:
https://patchwork.kernel.org/patch/10561557/
It didn't get applied because Paolo disagreed with having
our tools enforcing what our style guide says.
Personally I think we should just commit my patch, and then
we can stop having people manually pointing out where
submitters' patches don't match CODING_STYLE.
thanks
-- PMM
- [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Wainer dos Santos Moschetta, 2018/12/13
- [Qemu-devel] [PATCH 1/1] checkpatch: check for malformed comment block., Wainer dos Santos Moschetta, 2018/12/13
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Paolo Bonzini, 2018/12/13
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Peter Maydell, 2018/12/13
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Paolo Bonzini, 2018/12/13
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Markus Armbruster, 2018/12/14
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Paolo Bonzini, 2018/12/14
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Peter Maydell, 2018/12/14
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Markus Armbruster, 2018/12/14
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Peter Maydell, 2018/12/14
- Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Markus Armbruster, 2018/12/14
Re: [Qemu-devel] [PATCH 0/1] checkpatch: checker for comment block, Wainer dos Santos Moschetta, 2018/12/14