BUG_ON() Conversion in fs/dquot.c
authorEric Sesterhenn <[email protected]>
Sun, 2 Apr 2006 11:36:13 +0000 (13:36 +0200)
committerAdrian Bunk <[email protected]>
Sun, 2 Apr 2006 11:36:13 +0000 (13:36 +0200)
this changes if() BUG(); constructs to BUG_ON() which is
cleaner and can better optimized away

Signed-off-by: Eric Sesterhenn <[email protected]>
Signed-off-by: Adrian Bunk <[email protected]>
fs/dquot.c

index 6b388692093976c1661546b619678cc40d37e000..81d87a413c68c48c542f06c8bb5982f179f01072 100644 (file)
@@ -590,8 +590,7 @@ we_slept:
        atomic_dec(&dquot->dq_count);
 #ifdef __DQUOT_PARANOIA
        /* sanity check */
-       if (!list_empty(&dquot->dq_free))
-               BUG();
+       BUG_ON(!list_empty(&dquot->dq_free));
 #endif
        put_dquot_last(dquot);
        spin_unlock(&dq_list_lock);
@@ -666,8 +665,7 @@ we_slept:
                return NODQUOT;
        }
 #ifdef __DQUOT_PARANOIA
-       if (!dquot->dq_sb)      /* Has somebody invalidated entry under us? */
-               BUG();
+       BUG_ON(!dquot->dq_sb);  /* Has somebody invalidated entry under us? */
 #endif
 
        return dquot;