mbox series

[RESEND,00/13] trivial: Detect and remove superfluous semicolons in C code

Message ID 20200218094402.26625-1-philmd@redhat.com (mailing list archive)
Headers show
Series trivial: Detect and remove superfluous semicolons in C code | expand

Message

Philippe Mathieu-Daudé Feb. 18, 2020, 9:43 a.m. UTC
Luc noticed a superfluous trailing semicolon:
https://lists.gnu.org/archive/html/qemu-devel/2020-02/msg04593.html

Prevent that by modifying checkpatch.pl and clean the codebase.

Philippe Mathieu-Daudé (13):
  scripts/checkpatch.pl: Detect superfluous semicolon in C code
  audio/alsaaudio: Remove superfluous semicolons
  block: Remove superfluous semicolons
  block/io_uring: Remove superfluous semicolon
  hw/arm/xlnx-versal: Remove superfluous semicolon
  hw/m68k/next-cube: Remove superfluous semicolon
  hw/scsi/esp: Remove superfluous semicolon
  hw/vfio/display: Remove superfluous semicolon
  migration/multifd: Remove superfluous semicolon
  ui/input-barrier: Remove superfluous semicolon
  target/i386/whpx: Remove superfluous semicolon
  tests/qtest/libqos/qgraph: Remove superfluous semicolons
  contrib/rdmacm-mux: Remove superfluous semicolon

 audio/alsaaudio.c           | 4 ++--
 block.c                     | 4 ++--
 block/io_uring.c            | 2 +-
 contrib/rdmacm-mux/main.c   | 2 +-
 hw/arm/xlnx-versal-virt.c   | 2 +-
 hw/m68k/next-cube.c         | 2 +-
 hw/scsi/esp.c               | 2 +-
 hw/vfio/display.c           | 2 +-
 migration/multifd.c         | 2 +-
 target/i386/whpx-all.c      | 2 +-
 tests/qtest/libqos/qgraph.c | 4 ++--
 ui/input-barrier.c          | 2 +-
 scripts/checkpatch.pl       | 5 +++++
 13 files changed, 20 insertions(+), 15 deletions(-)

Comments

Paolo Bonzini Feb. 18, 2020, 5:04 p.m. UTC | #1
On 18/02/20 10:43, Philippe Mathieu-Daudé wrote:
> Luc noticed a superfluous trailing semicolon:
> https://lists.gnu.org/archive/html/qemu-devel/2020-02/msg04593.html
> 
> Prevent that by modifying checkpatch.pl and clean the codebase.
> 
> Philippe Mathieu-Daudé (13):
>   scripts/checkpatch.pl: Detect superfluous semicolon in C code
>   audio/alsaaudio: Remove superfluous semicolons
>   block: Remove superfluous semicolons
>   block/io_uring: Remove superfluous semicolon
>   hw/arm/xlnx-versal: Remove superfluous semicolon
>   hw/m68k/next-cube: Remove superfluous semicolon
>   hw/scsi/esp: Remove superfluous semicolon
>   hw/vfio/display: Remove superfluous semicolon
>   migration/multifd: Remove superfluous semicolon
>   ui/input-barrier: Remove superfluous semicolon
>   target/i386/whpx: Remove superfluous semicolon
>   tests/qtest/libqos/qgraph: Remove superfluous semicolons
>   contrib/rdmacm-mux: Remove superfluous semicolon
> 
>  audio/alsaaudio.c           | 4 ++--
>  block.c                     | 4 ++--
>  block/io_uring.c            | 2 +-
>  contrib/rdmacm-mux/main.c   | 2 +-
>  hw/arm/xlnx-versal-virt.c   | 2 +-
>  hw/m68k/next-cube.c         | 2 +-
>  hw/scsi/esp.c               | 2 +-
>  hw/vfio/display.c           | 2 +-
>  migration/multifd.c         | 2 +-
>  target/i386/whpx-all.c      | 2 +-
>  tests/qtest/libqos/qgraph.c | 4 ++--
>  ui/input-barrier.c          | 2 +-
>  scripts/checkpatch.pl       | 5 +++++
>  13 files changed, 20 insertions(+), 15 deletions(-)
> 

Acked-by: Paolo Bonzini <pbonzini@redhat.com>

Laurent, can you queue this in qemu-trivial?
Laurent Vivier Feb. 18, 2020, 6:54 p.m. UTC | #2
On 18/02/2020 18:04, Paolo Bonzini wrote:
> On 18/02/20 10:43, Philippe Mathieu-Daudé wrote:
>> Luc noticed a superfluous trailing semicolon:
>> https://lists.gnu.org/archive/html/qemu-devel/2020-02/msg04593.html
>>
>> Prevent that by modifying checkpatch.pl and clean the codebase.
>>
>> Philippe Mathieu-Daudé (13):
>>   scripts/checkpatch.pl: Detect superfluous semicolon in C code
>>   audio/alsaaudio: Remove superfluous semicolons
>>   block: Remove superfluous semicolons
>>   block/io_uring: Remove superfluous semicolon
>>   hw/arm/xlnx-versal: Remove superfluous semicolon
>>   hw/m68k/next-cube: Remove superfluous semicolon
>>   hw/scsi/esp: Remove superfluous semicolon
>>   hw/vfio/display: Remove superfluous semicolon
>>   migration/multifd: Remove superfluous semicolon
>>   ui/input-barrier: Remove superfluous semicolon
>>   target/i386/whpx: Remove superfluous semicolon
>>   tests/qtest/libqos/qgraph: Remove superfluous semicolons
>>   contrib/rdmacm-mux: Remove superfluous semicolon
>>
>>  audio/alsaaudio.c           | 4 ++--
>>  block.c                     | 4 ++--
>>  block/io_uring.c            | 2 +-
>>  contrib/rdmacm-mux/main.c   | 2 +-
>>  hw/arm/xlnx-versal-virt.c   | 2 +-
>>  hw/m68k/next-cube.c         | 2 +-
>>  hw/scsi/esp.c               | 2 +-
>>  hw/vfio/display.c           | 2 +-
>>  migration/multifd.c         | 2 +-
>>  target/i386/whpx-all.c      | 2 +-
>>  tests/qtest/libqos/qgraph.c | 4 ++--
>>  ui/input-barrier.c          | 2 +-
>>  scripts/checkpatch.pl       | 5 +++++
>>  13 files changed, 20 insertions(+), 15 deletions(-)
>>
> 
> Acked-by: Paolo Bonzini <pbonzini@redhat.com>
> 
> Laurent, can you queue this in qemu-trivial?
> 
Yes, I will.

Thanks,
Laurent
Laurent Vivier Feb. 18, 2020, 7:08 p.m. UTC | #3
Le 18/02/2020 à 18:04, Paolo Bonzini a écrit :
> On 18/02/20 10:43, Philippe Mathieu-Daudé wrote:
>> Luc noticed a superfluous trailing semicolon:
>> https://lists.gnu.org/archive/html/qemu-devel/2020-02/msg04593.html
>>
>> Prevent that by modifying checkpatch.pl and clean the codebase.
>>
>> Philippe Mathieu-Daudé (13):
>>   scripts/checkpatch.pl: Detect superfluous semicolon in C code
>>   audio/alsaaudio: Remove superfluous semicolons
>>   block: Remove superfluous semicolons
>>   block/io_uring: Remove superfluous semicolon
>>   hw/arm/xlnx-versal: Remove superfluous semicolon
>>   hw/m68k/next-cube: Remove superfluous semicolon
>>   hw/scsi/esp: Remove superfluous semicolon
>>   hw/vfio/display: Remove superfluous semicolon
>>   migration/multifd: Remove superfluous semicolon
>>   ui/input-barrier: Remove superfluous semicolon
>>   target/i386/whpx: Remove superfluous semicolon
>>   tests/qtest/libqos/qgraph: Remove superfluous semicolons
>>   contrib/rdmacm-mux: Remove superfluous semicolon
>>
>>  audio/alsaaudio.c           | 4 ++--
>>  block.c                     | 4 ++--
>>  block/io_uring.c            | 2 +-
>>  contrib/rdmacm-mux/main.c   | 2 +-
>>  hw/arm/xlnx-versal-virt.c   | 2 +-
>>  hw/m68k/next-cube.c         | 2 +-
>>  hw/scsi/esp.c               | 2 +-
>>  hw/vfio/display.c           | 2 +-
>>  migration/multifd.c         | 2 +-
>>  target/i386/whpx-all.c      | 2 +-
>>  tests/qtest/libqos/qgraph.c | 4 ++--
>>  ui/input-barrier.c          | 2 +-
>>  scripts/checkpatch.pl       | 5 +++++
>>  13 files changed, 20 insertions(+), 15 deletions(-)
>>
> 
> Acked-by: Paolo Bonzini <pbonzini@redhat.com>
> 
> Laurent, can you queue this in qemu-trivial?
> 

Queued, except patches 3, 4 (taken by Kevin) and 9 (by Juan)

Thanks,
Laurent