[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMj1kXGh+CzuXkAnqsoMO2A3T1p=D6uFOV347Ym5+VFn5U1gWg@mail.gmail.com>
Date: Tue, 29 Sep 2020 14:20:00 +0200
From: Ard Biesheuvel <ardb@...nel.org>
To: Julia Lawall <Julia.Lawall@...ia.fr>
Cc: linux-iio@...r.kernel.org, drbd-dev@...ts.linbit.com,
Valdis Klētnieks <valdis.kletnieks@...edu>,
David Lechner <david@...hnology.com>,
Neil Armstrong <narmstrong@...libre.com>,
Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
linux-wireless@...r.kernel.org,
"Rafael J. Wysocki" <rafael@...nel.org>,
kernel-janitors@...r.kernel.org,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
linux-stm32@...md-mailman.stormreply.com,
linux-block@...r.kernel.org, linux-ide@...r.kernel.org,
Linux Crypto Mailing List <linux-crypto@...r.kernel.org>,
Joe Perches <joe@...ches.com>,
linux-amlogic@...ts.infradead.org,
Thomas Gleixner <tglx@...utronix.de>,
ACPI Devel Maling List <linux-acpi@...r.kernel.org>,
openipmi-developer@...ts.sourceforge.net,
linux-clk@...r.kernel.org,
Linux ARM <linux-arm-kernel@...ts.infradead.org>,
Jerome Brunet <jbrunet@...libre.com>
Subject: Re: [PATCH 00/18] use semicolons rather than commas to separate statements
On Sun, 27 Sep 2020 at 21:56, Julia Lawall <Julia.Lawall@...ia.fr> wrote:
>
> These patches replace commas by semicolons.
Why?
> This was done using the
> Coccinelle semantic patch (http://coccinelle.lip6.fr/) shown below.
>
> This semantic patch ensures that commas inside for loop headers will not be
> transformed. It also doesn't touch macro definitions.
>
> Coccinelle ensures that braces are added as needed when a single-statement
> branch turns into a multi-statement one.
>
> This semantic patch has a few false positives, for variable delcarations
> such as:
>
> LIST_HEAD(x), *y;
>
> The semantic patch could be improved to avoid these, but for the moment
> they have been removed manually (2 occurrences).
>
> // <smpl>
> @initialize:ocaml@
> @@
>
> let infunction p =
> (* avoid macros *)
> (List.hd p).current_element <> "something_else"
>
> let combined p1 p2 =
> (List.hd p1).line_end = (List.hd p2).line ||
> (((List.hd p1).line_end < (List.hd p2).line) &&
> ((List.hd p1).col < (List.hd p2).col))
>
> @bad@
> statement S;
> declaration d;
> position p;
> @@
>
> S@p
> d
>
> // special cases where newlines are needed (hope for no more than 5)
> @@
> expression e1,e2;
> statement S;
> position p != bad.p;
> position p1;
> position p2 :
> script:ocaml(p1) { infunction p1 && combined p1 p2 };
> @@
>
> - e1@p1,@S@p e2@p2;
> + e1; e2;
>
> @@
> expression e1,e2;
> statement S;
> position p != bad.p;
> position p1;
> position p2 :
> script:ocaml(p1) { infunction p1 && combined p1 p2 };
> @@
>
> - e1@p1,@S@p e2@p2;
> + e1; e2;
>
> @@
> expression e1,e2;
> statement S;
> position p != bad.p;
> position p1;
> position p2 :
> script:ocaml(p1) { infunction p1 && combined p1 p2 };
> @@
>
> - e1@p1,@S@p e2@p2;
> + e1; e2;
>
> @@
> expression e1,e2;
> statement S;
> position p != bad.p;
> position p1;
> position p2 :
> script:ocaml(p1) { infunction p1 && combined p1 p2 };
> @@
>
> - e1@p1,@S@p e2@p2;
> + e1; e2;
>
> @@
> expression e1,e2;
> statement S;
> position p != bad.p;
> position p1;
> position p2 :
> script:ocaml(p1) { infunction p1 && combined p1 p2 };
> @@
>
> - e1@p1,@S@p e2@p2;
> + e1; e2;
>
> @r@
> expression e1,e2;
> statement S;
> position p != bad.p;
> @@
>
> e1 ,@S@p e2;
>
> @@
> expression e1,e2;
> position p1;
> position p2 :
> script:ocaml(p1) { infunction p1 && not(combined p1 p2) };
> statement S;
> position r.p;
> @@
>
> e1@p1
> -,@S@p
> +;
> e2@p2
> ... when any
> // </smpl>
>
> ---
>
> drivers/acpi/processor_idle.c | 4 +++-
> drivers/ata/pata_icside.c | 21 +++++++++++++--------
> drivers/base/regmap/regmap-debugfs.c | 2 +-
> drivers/bcma/driver_pci_host.c | 4 ++--
> drivers/block/drbd/drbd_receiver.c | 6 ++++--
> drivers/char/agp/amd-k7-agp.c | 2 +-
> drivers/char/agp/nvidia-agp.c | 2 +-
> drivers/char/agp/sworks-agp.c | 2 +-
> drivers/char/hw_random/iproc-rng200.c | 8 ++++----
> drivers/char/hw_random/mxc-rnga.c | 6 +++---
> drivers/char/hw_random/stm32-rng.c | 8 ++++----
> drivers/char/ipmi/bt-bmc.c | 6 +++---
> drivers/clk/meson/meson-aoclk.c | 2 +-
> drivers/clk/mvebu/ap-cpu-clk.c | 2 +-
> drivers/clk/uniphier/clk-uniphier-cpugear.c | 2 +-
> drivers/clk/uniphier/clk-uniphier-mux.c | 2 +-
> drivers/clocksource/mps2-timer.c | 6 +++---
> drivers/clocksource/timer-armada-370-xp.c | 8 ++++----
> drivers/counter/ti-eqep.c | 2 +-
> drivers/crypto/amcc/crypto4xx_alg.c | 2 +-
> drivers/crypto/atmel-tdes.c | 2 +-
> drivers/crypto/hifn_795x.c | 4 ++--
> drivers/crypto/talitos.c | 8 ++++----
> 23 files changed, 60 insertions(+), 51 deletions(-)
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@...ts.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Powered by blists - more mailing lists