On Tue, 14 May 2019, Markus Elfring wrote:
> From: Markus Elfring <[email protected]>
> Date: Tue, 14 May 2019 17:18:24 +0200
>
> A return statement was specified with a known value for three branches
> of a SmPL disjunction.
> Reduce duplicate SmPL code there by using another disjunction for
> these return values.
>
> Signed-off-by: Markus Elfring <[email protected]>
NACK. The goak is not to squeeze the most information into the fewest
number of characters. The rule was fine as it was.
julia
> ---
> scripts/coccinelle/free/pci_free_consistent.cocci | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/scripts/coccinelle/free/pci_free_consistent.cocci
> b/scripts/coccinelle/free/pci_free_consistent.cocci
> index 2056d6680cb8..45bc14ece151 100644
> --- a/scripts/coccinelle/free/pci_free_consistent.cocci
> +++ b/scripts/coccinelle/free/pci_free_consistent.cocci
> @@ -25,11 +25,11 @@ if (id == NULL || ...) { ... return ...; }
> when != e = (T)id
> when exists
> (
> -return 0;
> -|
> -return 1;
> -|
> -return id;
> + return
> +(0
> +|1
> +|id
> +);
> |
> return@p2 ...;
> )
> --
> 2.21.0
>
>
