Re: [PATCH 2/2] drivers/regulator/s5m8767.c: adjust duplicate test

2013-02-28 Thread Mark Brown
On Sun, Feb 24, 2013 at 12:55:34PM +0100, Julia Lawall wrote: > From: Julia Lawall > > Delete successive tests to the same location. Applied, thanks. If you're sending a bunch of patches intended to be applied separately it's probably not worth numbering them, it avoids confusion (like waiting

[PATCH 2/2] drivers/regulator/s5m8767.c: adjust duplicate test

2013-02-24 Thread Julia Lawall
From: Julia Lawall Delete successive tests to the same location. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @s exists@ local idexpression y; expression x,e; @@ *if ( \(x == NULL\|IS_ERR(x)\|y != 0\) ) { ... when forall