Re: [PATCH 2/9] Add yaffs allocator, bitmap and attrib source

[Jesper Juhl]

On Thu, 4 Nov 2010, cdhmanning@xxxxxxxxx wrote:
[snip]
> +void yaffs_free_raw_obj(struct yaffs_dev *dev, struct yaffs_obj *obj)
> +{
> +      
> +     struct yaffs_allocator *allocator = dev->allocator;
> +     
> +     if (!allocator)
> +             YBUG();
> +     else {
> +             /* Link into the free list. */
> +             obj->siblings.next = (struct list_head *)(allocator->free_objs);
> +             allocator->free_objs = obj;
> +             allocator->n_free_objects++;
> +     }
> +}
> +             
> +void yaffs_deinit_raw_tnodes_and_objs(struct yaffs_dev *dev)
> +{
> +     if (dev->allocator) {
> +             yaffs_deinit_raw_tnodes(dev);
> +             yaffs_deinit_raw_objs(dev);
> +     
> +             YFREE(dev->allocator);
> +             dev->allocator = NULL;
> +     } else
> +             YBUG();
> +}
> +             
> +void yaffs_init_raw_tnodes_and_objs(struct yaffs_dev *dev)
> +{
> +     struct yaffs_allocator *allocator;  
> +      
> +     if (!dev->allocator) {
> +             allocator = YMALLOC(sizeof(struct yaffs_allocator));
> +             if (allocator) {
> +                     dev->allocator = allocator;
> +                     yaffs_init_raw_tnodes(dev);
> +                     yaffs_init_raw_objs(dev);
> +             }
> +     } else
> +             YBUG();
> +}

Tiny style issue; Documentation/CodingStyle says:

"Do not unnecessarily use braces where a single statement will do.

if (condition)
        action();

This does not apply if one branch of a conditional statement is a single
statement. Use braces in both branches.

if (condition) {
        do_this();
        do_that();
} else {
        otherwise();
}
"

So, in the above, you should have curly braces around both the 'if' and 
'else' parts since one of them is more than one statement.



-- 
Jesper Juhl <jj@xxxxxxxxxxxxx>             http://www.chaosbits.net/
Plain text mails only, please      http://www.expita.com/nomime.html
Don't top-post  http://www.catb.org/~esr/jargon/html/T/top-post.html

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/