Re: [PATCH 4/8] fs/ext3/inode.c: drop if around WARN_ON

2012-11-06 Thread Jan Kara
On Sat 03-11-12 21:30:21, Julia Lawall wrote: > From: Julia Lawall > > Just use WARN_ON rather than an if containing only WARN_ON(1). > > A simplified version of the semantic patch that makes this transformation > is as follows: (http://coccinelle.lip6.fr/) > > // > @@ > expression e; > @@ > -

[PATCH 4/8] fs/ext3/inode.c: drop if around WARN_ON

2012-11-03 Thread Julia Lawall
From: Julia Lawall Just use WARN_ON rather than an if containing only WARN_ON(1). A simplified version of the semantic patch that makes this transformation is as follows: (http://coccinelle.lip6.fr/) // @@ expression e; @@ - if (e) WARN_ON(1); + WARN_ON(e); // Signed-off-by: Julia Lawall -