[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1312293344.4723.157.camel@crucifix>
Date: Tue, 02 Aug 2011 15:55:44 +0200
From: Jonas Bonn <jonas@...thpole.se>
To: Julia Lawall <julia@...u.dk>
Cc: trivial@...nel.org, kernel-janitors@...r.kernel.org,
Arnd Bergmann <arnd@...db.de>, linux@...ts.openrisc.net,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 10/11] arch/openrisc/mm/init.c: trivial: use BUG_ON
On Tue, 2011-08-02 at 12:35 +0200, Julia Lawall wrote:
> From: Julia Lawall <julia@...u.dk>
>
> Use BUG_ON(x) rather than if(x) BUG();
>
> The semantic patch that fixes this problem is as follows:
> (http://coccinelle.lip6.fr/)
>
> // <smpl>
> @@ identifier x; @@
> -if (x) BUG();
> +BUG_ON(x);
>
> @@ identifier x; @@
> -if (!x) BUG();
> +BUG_ON(!x);
> // </smpl>
>
> Signed-off-by: Julia Lawall <julia@...u.dk>
>
Will apply. Thanks.
Best regards,
Jonas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists