BUG_ON() Conversion in fs/hfsplus/
authorEric Sesterhenn <snakebyte@gmx.de>
Fri, 31 Mar 2006 23:14:43 +0000 (01:14 +0200)
committerAdrian Bunk <bunk@stusta.de>
Fri, 31 Mar 2006 23:14:43 +0000 (01:14 +0200)
this changes if() BUG(); constructs to BUG_ON() which is
cleaner, contains unlikely() and can better optimized away.

Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Adrian Bunk <bunk@stusta.de>
fs/hfsplus/bnode.c
fs/hfsplus/btree.c

index 8f07e8fbd03d34cdf4b294d1395234f9ff328546..746abc9ecf70f344418c37c8f47062aeb9c5e3ef 100644 (file)
@@ -466,8 +466,7 @@ void hfs_bnode_unhash(struct hfs_bnode *node)
        for (p = &node->tree->node_hash[hfs_bnode_hash(node->this)];
             *p && *p != node; p = &(*p)->next_hash)
                ;
-       if (!*p)
-               BUG();
+       BUG_ON(!*p);
        *p = node->next_hash;
        node->tree->node_hash_cnt--;
 }
@@ -622,8 +621,7 @@ void hfs_bnode_put(struct hfs_bnode *node)
 
                dprint(DBG_BNODE_REFS, "put_node(%d:%d): %d\n",
                       node->tree->cnid, node->this, atomic_read(&node->refcnt));
-               if (!atomic_read(&node->refcnt))
-                       BUG();
+               BUG_ON(!atomic_read(&node->refcnt));
                if (!atomic_dec_and_lock(&node->refcnt, &tree->hash_lock))
                        return;
                for (i = 0; i < tree->pages_per_bnode; i++) {
index a67edfa34e9ec6ac847d9e2fe2f6bb08725e038d..effa8991999c9d0d140939f1a81fe8ca638eb8f6 100644 (file)
@@ -269,8 +269,7 @@ void hfs_bmap_free(struct hfs_bnode *node)
        u8 *data, byte, m;
 
        dprint(DBG_BNODE_MOD, "btree_free_node: %u\n", node->this);
-       if (!node->this)
-               BUG();
+       BUG_ON(!node->this);
        tree = node->tree;
        nidx = node->this;
        node = hfs_bnode_find(tree, 0);