Messages in this thread Patch in this message |  | | | From | Julia Lawall <> | | Subject | [PATCH 10/11] arch/openrisc/mm/init.c: trivial: use BUG_ON | | Date | Tue, 2 Aug 2011 12:35:03 +0200 |
| |
From: Julia Lawall <julia@diku.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@diku.dk>
--- arch/openrisc/mm/init.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff -u -p a/arch/openrisc/mm/init.c b/arch/openrisc/mm/init.c --- a/arch/openrisc/mm/init.c +++ b/arch/openrisc/mm/init.c @@ -222,8 +222,7 @@ void __init mem_init(void) { int codesize, reservedpages, datasize, initsize; - if (!mem_map) - BUG(); + BUG_ON(!mem_map); set_max_mapnr_init();
|  |