yaffs Clean up conditional bracing

If a conditional has braces then all paths have braces even if they only
have one statement.

Signed-off-by: Charles Manning <cdhmanning@gmail.com>

diff --git a/yaffs_allocator.c b/yaffs_allocator.c
index 03ede863744809989918ac984065a29abb7fb699..b9fe31e70d475840e8c613d8eb2a2fa29663913e 100644 (file)
--- a/yaffs_allocator.c
+++ b/yaffs_allocator.c
@@ -123,8 +123,9 @@ static void yaffs_init_raw_tnodes(struct yaffs_dev *dev)
                allocator->free_tnodes = NULL;
                allocator->n_free_tnodes = 0;
                allocator->n_tnodes_created = 0;
                allocator->free_tnodes = NULL;
                allocator->n_free_tnodes = 0;
                allocator->n_tnodes_created = 0;

-       } else
+       } else {

                YBUG();
                YBUG();

+       }

 }
 
 static int yaffs_create_tnodes(struct yaffs_dev *dev, int n_tnodes)
 }
 
 static int yaffs_create_tnodes(struct yaffs_dev *dev, int n_tnodes)


@@ -243,8 +244,9 @@ static void yaffs_init_raw_objs(struct yaffs_dev *dev)
                allocator->allocated_obj_list = NULL;
                allocator->free_objs = NULL;
                allocator->n_free_objects = 0;
                allocator->allocated_obj_list = NULL;
                allocator->free_objs = NULL;
                allocator->n_free_objects = 0;

-       } else
+       } else {

                YBUG();
                YBUG();

+       }

 }
 
 static void yaffs_deinit_raw_objs(struct yaffs_dev *dev)
 }
 
 static void yaffs_deinit_raw_objs(struct yaffs_dev *dev)


@@ -371,8 +373,9 @@ void yaffs_deinit_raw_tnodes_and_objs(struct yaffs_dev *dev)
 
                YFREE(dev->allocator);
                dev->allocator = NULL;
 
                YFREE(dev->allocator);
                dev->allocator = NULL;

-       } else
+       } else {

                YBUG();
                YBUG();

+       }

 }
 
 void yaffs_init_raw_tnodes_and_objs(struct yaffs_dev *dev)
 }
 
 void yaffs_init_raw_tnodes_and_objs(struct yaffs_dev *dev)


@@ -386,8 +389,9 @@ void yaffs_init_raw_tnodes_and_objs(struct yaffs_dev *dev)
                        yaffs_init_raw_tnodes(dev);
                        yaffs_init_raw_objs(dev);
                }
                        yaffs_init_raw_tnodes(dev);
                        yaffs_init_raw_objs(dev);
                }

-       } else
+       } else {

                YBUG();
                YBUG();

+       }

 }
 
 #endif
 }
 
 #endif

diff --git a/yaffs_checkptrw.c b/yaffs_checkptrw.c
index 2cb144c2f4831e4770cf54ce19c65c5d9bac6571..02b8ce6a0f2e088f8a43709f54796873d8c0f418 100644 (file)
--- a/yaffs_checkptrw.c
+++ b/yaffs_checkptrw.c
@@ -403,8 +403,9 @@ int yaffs_checkpt_close(struct yaffs_dev *dev)
                YFREE(dev->checkpt_buffer);
                dev->checkpt_buffer = NULL;
                return 1;
                YFREE(dev->checkpt_buffer);
                dev->checkpt_buffer = NULL;
                return 1;

-       } else
+       } else {

                return 0;
                return 0;

+        }

 }
 
 int yaffs2_checkpt_invalidate_stream(struct yaffs_dev *dev)
 }
 
 int yaffs2_checkpt_invalidate_stream(struct yaffs_dev *dev)

diff --git a/yaffs_guts.c b/yaffs_guts.c
index a9fc46c0a4814a2a03697c985a074cf2e1d393d0..53c2ff6349360266ee2302d00259e32b5bbb47a6 100644 (file)
--- a/yaffs_guts.c
+++ b/yaffs_guts.c
@@ -1708,8 +1708,9 @@ static int yaffs_init_blocks(struct yaffs_dev *dev)
                dev->block_info =
                    YMALLOC_ALT(n_blocks * sizeof(struct yaffs_block_info));
                dev->block_info_alt = 1;
                dev->block_info =
                    YMALLOC_ALT(n_blocks * sizeof(struct yaffs_block_info));
                dev->block_info_alt = 1;

-       } else
+       } else {

                dev->block_info_alt = 0;
                dev->block_info_alt = 0;

+        }

 
        if (dev->block_info) {
                /* Set up dynamic blockinfo stuff. */
 
        if (dev->block_info) {
                /* Set up dynamic blockinfo stuff. */


@@ -1719,8 +1720,9 @@ static int yaffs_init_blocks(struct yaffs_dev *dev)
                        dev->chunk_bits =
                            YMALLOC_ALT(dev->chunk_bit_stride * n_blocks);
                        dev->chunk_bits_alt = 1;
                        dev->chunk_bits =
                            YMALLOC_ALT(dev->chunk_bit_stride * n_blocks);
                        dev->chunk_bits_alt = 1;

-               } else
+               } else {

                        dev->chunk_bits_alt = 0;
                        dev->chunk_bits_alt = 0;

+                }

        }
 
        if (dev->block_info && dev->chunk_bits) {
        }
 
        if (dev->block_info && dev->chunk_bits) {


@@ -2174,12 +2176,13 @@ static int yaffs_gc_block(struct yaffs_dev *dev, int block, int whole_block)
                                                                          oh,
                                                                          &tags,
                                                                          1);
                                                                          oh,
                                                                          &tags,
                                                                          1);

-                                       } else
+                                       } else {

                                                new_chunk =
                                                    yaffs_write_new_chunk(dev,
                                                                          buffer,
                                                                          &tags,
                                                                          1);
                                                new_chunk =
                                                    yaffs_write_new_chunk(dev,
                                                                          buffer,
                                                                          &tags,
                                                                          1);

+                                        }

 
                                        if (new_chunk < 0) {
                                                ret_val = YAFFS_FAIL;
 
                                        if (new_chunk < 0) {
                                                ret_val = YAFFS_FAIL;


@@ -2385,8 +2388,9 @@ static unsigned yaffs_find_gc_block(struct yaffs_dev *dev,
                        bi = yaffs_get_block_info(dev, selected);
                        dev->gc_pages_in_use =
                            bi->pages_in_use - bi->soft_del_pages;
                        bi = yaffs_get_block_info(dev, selected);
                        dev->gc_pages_in_use =
                            bi->pages_in_use - bi->soft_del_pages;

-               } else
+               } else {

                        dev->gc_not_done = 0;
                        dev->gc_not_done = 0;

+                }

        }
 
        if (selected) {
        }
 
        if (selected) {


@@ -2921,8 +2925,9 @@ int yaffs_update_oh(struct yaffs_obj *in, const YCHAR * name, int force,
 
                        memcpy(old_name, oh->name, sizeof(oh->name));
                        memset(buffer, 0xFF, sizeof(struct yaffs_obj_hdr));
 
                        memcpy(old_name, oh->name, sizeof(oh->name));
                        memset(buffer, 0xFF, sizeof(struct yaffs_obj_hdr));

-               } else
+               } else {

                        memset(buffer, 0xFF, dev->data_bytes_per_chunk);
                        memset(buffer, 0xFF, dev->data_bytes_per_chunk);

+                }

 
                oh->type = in->variant_type;
                oh->yst_mode = in->yst_mode;
 
                oh->type = in->variant_type;
                oh->yst_mode = in->yst_mode;


@@ -2938,10 +2943,11 @@ int yaffs_update_oh(struct yaffs_obj *in, const YCHAR * name, int force,
                if (name && *name) {
                        memset(oh->name, 0, sizeof(oh->name));
                        yaffs_load_oh_from_name(dev, oh->name, name);
                if (name && *name) {
                        memset(oh->name, 0, sizeof(oh->name));
                        yaffs_load_oh_from_name(dev, oh->name, name);

-               } else if (prev_chunk_id > 0)
+               } else if (prev_chunk_id > 0) {

                        memcpy(oh->name, old_name, sizeof(oh->name));
                        memcpy(oh->name, old_name, sizeof(oh->name));

-               else
+               } else {

                        memset(oh->name, 0, sizeof(oh->name));
                        memset(oh->name, 0, sizeof(oh->name));

+                }

 
                oh->is_shrink = is_shrink;
 
 
                oh->is_shrink = is_shrink;
 


@@ -3210,9 +3216,9 @@ static struct yaffs_cache *yaffs_grab_chunk_cache(struct yaffs_dev *dev)
 
                }
                return cache;
 
                }
                return cache;

-       } else
+       } else {

                return NULL;
                return NULL;

-
+        }

 }
 
 /* Find a cached chunk */
 }
 
 /* Find a cached chunk */


@@ -3932,11 +3938,12 @@ static int yaffs_unlink_worker(struct yaffs_obj *obj)
                default:
                        return YAFFS_FAIL;
                }
                default:
                        return YAFFS_FAIL;
                }

-       } else if (yaffs_is_non_empty_dir(obj))
+       } else if (yaffs_is_non_empty_dir(obj)) {

                return YAFFS_FAIL;
                return YAFFS_FAIL;

-       else
+       } else {

                return yaffs_change_obj_name(obj, obj->my_dev->unlinked_dir,
                                             _Y("unlinked"), 0, 0);
                return yaffs_change_obj_name(obj, obj->my_dev->unlinked_dir,
                                             _Y("unlinked"), 0, 0);

+        }

 }
 
 static int yaffs_unlink_obj(struct yaffs_obj *obj)
 }
 
 static int yaffs_unlink_obj(struct yaffs_obj *obj)


@@ -4103,11 +4110,11 @@ static void yaffs_fix_hanging_objs(struct yaffs_dev *dev)
                                        hanging = 0;
                                } else if (!parent
                                           || parent->variant_type !=
                                        hanging = 0;
                                } else if (!parent
                                           || parent->variant_type !=

-                                          YAFFS_OBJECT_TYPE_DIRECTORY)
+                                          YAFFS_OBJECT_TYPE_DIRECTORY) {

                                        hanging = 1;
                                        hanging = 1;

-                               else if (yaffs_has_null_parent(dev, parent))
+                               } else if (yaffs_has_null_parent(dev, parent)) {

                                        hanging = 0;
                                        hanging = 0;

-                               else {
+                               } else {

                                        /*
                                         * Need to follow the parent chain to see if it is hanging.
                                         */
                                        /*
                                         * Need to follow the parent chain to see if it is hanging.
                                         */


@@ -4248,8 +4255,9 @@ static void yaffs_update_parent(struct yaffs_obj *obj)
                           obj->obj_id));
                }
 
                           obj->obj_id));
                }
 

-       } else
+       } else {

                yaffs_update_oh(obj, NULL, 0, 0, 0, NULL);
                yaffs_update_oh(obj, NULL, 0, 0, 0, NULL);

+        }

 }
 
 void yaffs_update_dirty_dirs(struct yaffs_dev *dev)
 }
 
 void yaffs_update_dirty_dirs(struct yaffs_dev *dev)


@@ -4378,8 +4386,7 @@ struct yaffs_obj *yaffs_find_by_name(struct yaffs_obj *directory,
 
                        /* Special case for lost-n-found */
                        if (l->obj_id == YAFFS_OBJECTID_LOSTNFOUND) {
 
                        /* Special case for lost-n-found */
                        if (l->obj_id == YAFFS_OBJECTID_LOSTNFOUND) {

-                               if (yaffs_strcmp(name, YAFFS_LOSTNFOUND_NAME) ==
-                                   0)
+                               if (!yaffs_strcmp(name, YAFFS_LOSTNFOUND_NAME))

                                        return l;
                        } else if (yaffs_sum_cmp(l->sum, sum)
                                   || l->hdr_chunk <= 0) {
                                        return l;
                        } else if (yaffs_sum_cmp(l->sum, sum)
                                   || l->hdr_chunk <= 0) {


@@ -4468,11 +4475,15 @@ static void yaffs_load_name_from_oh(struct yaffs_dev *dev, YCHAR * name,
                                ascii_oh_name++;
                                n--;
                        }
                                ascii_oh_name++;
                                n--;
                        }

-               } else
+               } else {

                        yaffs_strncpy(name, oh_name + 1, buff_size - 1);
                        yaffs_strncpy(name, oh_name + 1, buff_size - 1);

-       } else
+                }
+       } else {
+#else
+        {

 #endif
                yaffs_strncpy(name, oh_name, buff_size - 1);
 #endif
                yaffs_strncpy(name, oh_name, buff_size - 1);

+        }

 }
 
 static void yaffs_load_oh_from_name(struct yaffs_dev *dev, YCHAR * oh_name,
 }
 
 static void yaffs_load_oh_from_name(struct yaffs_dev *dev, YCHAR * oh_name,


@@ -4511,9 +4522,12 @@ static void yaffs_load_oh_from_name(struct yaffs_dev *dev, YCHAR * oh_name,
                        yaffs_strncpy(oh_name + 1, name,
                                      YAFFS_MAX_NAME_LENGTH - 2);
                }
                        yaffs_strncpy(oh_name + 1, name,
                                      YAFFS_MAX_NAME_LENGTH - 2);
                }

-       } else
+       } else {
+#else
+        {

 #endif
                yaffs_strncpy(oh_name, name, YAFFS_MAX_NAME_LENGTH - 1);
 #endif
                yaffs_strncpy(oh_name, name, YAFFS_MAX_NAME_LENGTH - 1);

+        }

 
 }
 
 
 }
 


@@ -4926,8 +4940,9 @@ int yaffs_guts_initialise(struct yaffs_dev *dev)
                        dev->tnode_width = 16;
                else
                        dev->tnode_width = bits;
                        dev->tnode_width = 16;
                else
                        dev->tnode_width = bits;

-       } else
+       } else {

                dev->tnode_width = 16;
                dev->tnode_width = 16;

+        }

 
        dev->tnode_mask = (1 << dev->tnode_width) - 1;
 
 
        dev->tnode_mask = (1 << dev->tnode_width) - 1;
 


@@ -5078,8 +5093,9 @@ int yaffs_guts_initialise(struct yaffs_dev *dev)
                                if (!init_failed && !yaffs2_scan_backwards(dev))
                                        init_failed = 1;
                        }
                                if (!init_failed && !yaffs2_scan_backwards(dev))
                                        init_failed = 1;
                        }

-               } else if (!yaffs1_scan(dev))
+               } else if (!yaffs1_scan(dev)) {

                        init_failed = 1;
                        init_failed = 1;

+                }

 
                yaffs_strip_deleted_objs(dev);
                yaffs_fix_hanging_objs(dev);
 
                yaffs_strip_deleted_objs(dev);
                yaffs_fix_hanging_objs(dev);

diff --git a/yaffs_mtdif2_multi.c b/yaffs_mtdif2_multi.c
index 555f937cebf58d91b86d47de33010dd9fe6c9d38..26609f7a3995efa167010a5fdff3e7934b5c5eda 100644 (file)
--- a/yaffs_mtdif2_multi.c
+++ b/yaffs_mtdif2_multi.c
@@ -71,8 +71,9 @@ int nandmtd2_write_chunk_tags(struct yaffs_dev *dev, int nand_chunk,
                                                            dev->
                                                            data_bytes_per_chunk);
                yaffs_pack_tags2_tags_only(pt2tp, tags);
                                                            dev->
                                                            data_bytes_per_chunk);
                yaffs_pack_tags2_tags_only(pt2tp, tags);

-       } else
+       } else {

                yaffs_pack_tags2(&pt, tags, !dev->param.no_tags_ecc);
                yaffs_pack_tags2(&pt, tags, !dev->param.no_tags_ecc);

+        }

 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
        ops.mode = MTD_OOB_AUTO;
 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
        ops.mode = MTD_OOB_AUTO;

diff --git a/yaffs_mtdif2_single.c b/yaffs_mtdif2_single.c
index 00040cd9017eb81092fdd705451abea8f4fcb57b..0835078fe66c462ca0657a80b5435887052118c4 100644 (file)
--- a/yaffs_mtdif2_single.c
+++ b/yaffs_mtdif2_single.c
@@ -67,8 +67,9 @@ int nandmtd2_write_chunk_tags(struct yaffs_dev *dev, int nand_chunk,
                                                            dev->
                                                            data_bytes_per_chunk);
                yaffs_pack_tags2_tags_only(pt2tp, tags);
                                                            dev->
                                                            data_bytes_per_chunk);
                yaffs_pack_tags2_tags_only(pt2tp, tags);

-       } else
+       } else {

                yaffs_pack_tags2(&pt, tags, !dev->param.no_tags_ecc);
                yaffs_pack_tags2(&pt, tags, !dev->param.no_tags_ecc);

+        }

 
        ops.mode = MTD_OOB_AUTO;
        ops.ooblen = (dev->param.inband_tags) ? 0 : packed_tags_size;
 
        ops.mode = MTD_OOB_AUTO;
        ops.ooblen = (dev->param.inband_tags) ? 0 : packed_tags_size;

diff --git a/yaffs_nameval.c b/yaffs_nameval.c
index d7cb72aa00a6bf94ea9d275f688e6b879e14ea93..d8c548a1b832e3c10b7c7ed1cc45d6e66bf29bac 100644 (file)
--- a/yaffs_nameval.c
+++ b/yaffs_nameval.c
@@ -81,8 +81,9 @@ int nval_del(char *xb, int xb_size, const YCHAR * name)
                memcpy(xb + pos, xb + pos + size, xb_size - (pos + size));
                memset(xb + (xb_size - size), 0, size);
                return 0;
                memcpy(xb + pos, xb + pos + size, xb_size - (pos + size));
                memset(xb + (xb_size - size), 0, size);
                return 0;

-       } else
+       } else {

                return -ENODATA;
                return -ENODATA;

+        }

 }
 
 int nval_set(char *xb, int xb_size, const YCHAR * name, const char *buf,
 }
 
 int nval_set(char *xb, int xb_size, const YCHAR * name, const char *buf,


@@ -182,8 +183,9 @@ int nval_list(const char *xb, int xb_size, char *buf, int bsize)
                                buf++;
                        }
                        ncopied += (name_len + 1);
                                buf++;
                        }
                        ncopied += (name_len + 1);

-               } else
+               } else {

                        filled = 1;
                        filled = 1;

+                }

                pos += size;
                if (pos < xb_size - sizeof(int))
                        memcpy(&size, xb + pos, sizeof(int));
                pos += size;
                if (pos < xb_size - sizeof(int))
                        memcpy(&size, xb + pos, sizeof(int));

diff --git a/yaffs_vfs_multi.c b/yaffs_vfs_multi.c
index 9e4e31a8c78597ad1e4095f4ca420803dbb86f96..0bb06f41dd01d2ebdf80c9f0fc57eb57476007c3 100644 (file)
--- a/yaffs_vfs_multi.c
+++ b/yaffs_vfs_multi.c
@@ -2334,11 +2334,13 @@ static int yaffs_bg_thread_fn(void *data)
                                        next_gc = now + HZ / 10 + 1;
                                else
                                        next_gc = now + HZ * 2;
                                        next_gc = now + HZ / 10 + 1;
                                else
                                        next_gc = now + HZ * 2;

-                       } else  /*
+                       } else  {
+                               /*

                                 * gc not running so set to next_dir_update
                                 * to cut down on wake ups
                                 */
                                next_gc = next_dir_update;
                                 * gc not running so set to next_dir_update
                                 * to cut down on wake ups
                                 */
                                next_gc = next_dir_update;

+                        }

                }
                yaffs_gross_unlock(dev);
 #if 1
                }
                yaffs_gross_unlock(dev);
 #if 1


@@ -2605,9 +2607,9 @@ static int yaffs_parse_options(struct yaffs_options *options,
                        options_str++;
                }
 
                        options_str++;
                }
 

-               if (!strcmp(cur_opt, "inband-tags"))
+               if (!strcmp(cur_opt, "inband-tags")) {

                        options->inband_tags = 1;
                        options->inband_tags = 1;

-               else if (!strcmp(cur_opt, "tags-ecc-off")) {
+               } else if (!strcmp(cur_opt, "tags-ecc-off")) {

                        options->tags_ecc_on = 0;
                        options->tags_ecc_overridden = 1;
                } else if (!strcmp(cur_opt, "tags-ecc-on")) {
                        options->tags_ecc_on = 0;
                        options->tags_ecc_overridden = 1;
                } else if (!strcmp(cur_opt, "tags-ecc-on")) {


@@ -2625,13 +2627,13 @@ static int yaffs_parse_options(struct yaffs_options *options,
                } else if (!strcmp(cur_opt, "empty-lost-and-found-on")) {
                        options->empty_lost_and_found = 1;
                        options->empty_lost_and_found_overridden = 1;
                } else if (!strcmp(cur_opt, "empty-lost-and-found-on")) {
                        options->empty_lost_and_found = 1;
                        options->empty_lost_and_found_overridden = 1;

-               } else if (!strcmp(cur_opt, "no-cache"))
+               } else if (!strcmp(cur_opt, "no-cache")) {

                        options->no_cache = 1;
                        options->no_cache = 1;

-               else if (!strcmp(cur_opt, "no-checkpoint-read"))
+               } else if (!strcmp(cur_opt, "no-checkpoint-read")) {

                        options->skip_checkpoint_read = 1;
                        options->skip_checkpoint_read = 1;

-               else if (!strcmp(cur_opt, "no-checkpoint-write"))
+               } else if (!strcmp(cur_opt, "no-checkpoint-write")) {

                        options->skip_checkpoint_write = 1;
                        options->skip_checkpoint_write = 1;

-               else if (!strcmp(cur_opt, "no-checkpoint")) {
+               } else if (!strcmp(cur_opt, "no-checkpoint")) {

                        options->skip_checkpoint_read = 1;
                        options->skip_checkpoint_write = 1;
                } else {
                        options->skip_checkpoint_read = 1;
                        options->skip_checkpoint_write = 1;
                } else {


@@ -3268,8 +3270,9 @@ static int yaffs_proc_read(char *page,
                                    sprintf(buf, "\nDevice %d \"%s\"\n", n,
                                            dev->param.name);
                                buf = yaffs_dump_dev_part0(buf, dev);
                                    sprintf(buf, "\nDevice %d \"%s\"\n", n,
                                            dev->param.name);
                                buf = yaffs_dump_dev_part0(buf, dev);

-                       } else
+                       } else {

                                buf = yaffs_dump_dev_part1(buf, dev);
                                buf = yaffs_dump_dev_part1(buf, dev);

+                        }

 
                        break;
                }
 
                        break;
                }


@@ -3489,8 +3492,9 @@ static int __init init_yaffs_fs(void)
                my_proc_entry->write_proc = yaffs_proc_write;
                my_proc_entry->read_proc = yaffs_proc_read;
                my_proc_entry->data = NULL;
                my_proc_entry->write_proc = yaffs_proc_write;
                my_proc_entry->read_proc = yaffs_proc_read;
                my_proc_entry->data = NULL;

-       } else
+       } else {

                return -ENOMEM;
                return -ENOMEM;

+        }

 
        debug_proc_entry = create_proc_entry("yaffs_stats",
                                             S_IRUGO | S_IFREG, YPROC_ROOT);
 
        debug_proc_entry = create_proc_entry("yaffs_stats",
                                             S_IRUGO | S_IFREG, YPROC_ROOT);


@@ -3499,8 +3503,9 @@ static int __init init_yaffs_fs(void)
                debug_proc_entry->write_proc = NULL;
                debug_proc_entry->read_proc = yaffs_stats_proc_read;
                debug_proc_entry->data = NULL;
                debug_proc_entry->write_proc = NULL;
                debug_proc_entry->read_proc = yaffs_stats_proc_read;
                debug_proc_entry->data = NULL;

-       } else
+       } else {

                return -ENOMEM;
                return -ENOMEM;

+        }

 
        /* Now add the file system entries */
 
 
        /* Now add the file system entries */
 

diff --git a/yaffs_vfs_single.c b/yaffs_vfs_single.c
index a17262cef01b30628f7ad09bd4574a7ada389b19..146aaef57e0e5417b2bda0df748f8890e0116b7e 100644 (file)
--- a/yaffs_vfs_single.c
+++ b/yaffs_vfs_single.c
@@ -1890,11 +1890,13 @@ static int yaffs_bg_thread_fn(void *data)
                                        next_gc = now + HZ / 10 + 1;
                                else
                                        next_gc = now + HZ * 2;
                                        next_gc = now + HZ / 10 + 1;
                                else
                                        next_gc = now + HZ * 2;

-                       } else  /*
+                       } else  {
+                               /*

                                 * gc not running so set to next_dir_update
                                 * to cut down on wake ups
                                 */
                                next_gc = next_dir_update;
                                 * gc not running so set to next_dir_update
                                 * to cut down on wake ups
                                 */
                                next_gc = next_dir_update;

+                        }

                }
                yaffs_gross_unlock(dev);
                expires = next_dir_update;
                }
                yaffs_gross_unlock(dev);
                expires = next_dir_update;


@@ -2100,9 +2102,9 @@ static int yaffs_parse_options(struct yaffs_options *options,
                        options_str++;
                }
 
                        options_str++;
                }
 

-               if (!strcmp(cur_opt, "inband-tags"))
+               if (!strcmp(cur_opt, "inband-tags")) {

                        options->inband_tags = 1;
                        options->inband_tags = 1;

-               else if (!strcmp(cur_opt, "tags-ecc-off")) {
+               } else if (!strcmp(cur_opt, "tags-ecc-off")) {

                        options->tags_ecc_on = 0;
                        options->tags_ecc_overridden = 1;
                } else if (!strcmp(cur_opt, "tags-ecc-on")) {
                        options->tags_ecc_on = 0;
                        options->tags_ecc_overridden = 1;
                } else if (!strcmp(cur_opt, "tags-ecc-on")) {


@@ -2120,13 +2122,13 @@ static int yaffs_parse_options(struct yaffs_options *options,
                } else if (!strcmp(cur_opt, "empty-lost-and-found-on")) {
                        options->empty_lost_and_found = 1;
                        options->empty_lost_and_found_overridden = 1;
                } else if (!strcmp(cur_opt, "empty-lost-and-found-on")) {
                        options->empty_lost_and_found = 1;
                        options->empty_lost_and_found_overridden = 1;

-               } else if (!strcmp(cur_opt, "no-cache"))
+               } else if (!strcmp(cur_opt, "no-cache")) {

                        options->no_cache = 1;
                        options->no_cache = 1;

-               else if (!strcmp(cur_opt, "no-checkpoint-read"))
+               } else if (!strcmp(cur_opt, "no-checkpoint-read")) {

                        options->skip_checkpoint_read = 1;
                        options->skip_checkpoint_read = 1;

-               else if (!strcmp(cur_opt, "no-checkpoint-write"))
+               } else if (!strcmp(cur_opt, "no-checkpoint-write")) {

                        options->skip_checkpoint_write = 1;
                        options->skip_checkpoint_write = 1;

-               else if (!strcmp(cur_opt, "no-checkpoint")) {
+               } else if (!strcmp(cur_opt, "no-checkpoint")) {

                        options->skip_checkpoint_read = 1;
                        options->skip_checkpoint_write = 1;
                } else {
                        options->skip_checkpoint_read = 1;
                        options->skip_checkpoint_write = 1;
                } else {


@@ -2693,8 +2695,9 @@ static int yaffs_proc_read(char *page,
                                    sprintf(buf, "\nDevice %d \"%s\"\n", n,
                                            dev->param.name);
                                buf = yaffs_dump_dev_part0(buf, dev);
                                    sprintf(buf, "\nDevice %d \"%s\"\n", n,
                                            dev->param.name);
                                buf = yaffs_dump_dev_part0(buf, dev);

-                       } else
+                       } else {

                                buf = yaffs_dump_dev_part1(buf, dev);
                                buf = yaffs_dump_dev_part1(buf, dev);

+                        }

 
                        break;
                }
 
                        break;
                }


@@ -2914,8 +2917,9 @@ static int __init init_yaffs_fs(void)
                my_proc_entry->write_proc = yaffs_proc_write;
                my_proc_entry->read_proc = yaffs_proc_read;
                my_proc_entry->data = NULL;
                my_proc_entry->write_proc = yaffs_proc_write;
                my_proc_entry->read_proc = yaffs_proc_read;
                my_proc_entry->data = NULL;

-       } else
+       } else {

                return -ENOMEM;
                return -ENOMEM;

+        }

 
        debug_proc_entry = create_proc_entry("yaffs_stats",
                                             S_IRUGO | S_IFREG, YPROC_ROOT);
 
        debug_proc_entry = create_proc_entry("yaffs_stats",
                                             S_IRUGO | S_IFREG, YPROC_ROOT);


@@ -2924,8 +2928,9 @@ static int __init init_yaffs_fs(void)
                debug_proc_entry->write_proc = NULL;
                debug_proc_entry->read_proc = yaffs_stats_proc_read;
                debug_proc_entry->data = NULL;
                debug_proc_entry->write_proc = NULL;
                debug_proc_entry->read_proc = yaffs_stats_proc_read;
                debug_proc_entry->data = NULL;

-       } else
+       } else {

                return -ENOMEM;
                return -ENOMEM;

+        }

 
        /* Now add the file system entries */
 
 
        /* Now add the file system entries */
 

diff --git a/yaffs_yaffs1.c b/yaffs_yaffs1.c
index 4ab0f22bb563959febbd22f0715237482f14ff60..b6c43e5ef3571c88c4e8a25b1698fa4efa372bee 100644 (file)
--- a/yaffs_yaffs1.c
+++ b/yaffs_yaffs1.c
@@ -303,8 +303,7 @@ int yaffs1_scan(struct yaffs_dev *dev)
                                                               variant.dir_variant.children);
                                        } else if (!parent
                                                   || parent->variant_type !=
                                                               variant.dir_variant.children);
                                        } else if (!parent
                                                   || parent->variant_type !=

-                                                  YAFFS_OBJECT_TYPE_DIRECTORY)
-                                       {
+                                                  YAFFS_OBJECT_TYPE_DIRECTORY) {

                                                /* Hoosterman, another problem....
                                                 * We're trying to use a non-directory as a directory
                                                 */
                                                /* Hoosterman, another problem....
                                                 * We're trying to use a non-directory as a directory
                                                 */

diff --git a/yaffs_yaffs2.c b/yaffs_yaffs2.c
index 97dac454f34cf3b8ed4649eae8e8f56a1d2b3e52..ad35fd5bb91468fcbfcac947a3f038f52777c882 100644 (file)
--- a/yaffs_yaffs2.c
+++ b/yaffs_yaffs2.c
@@ -536,8 +536,9 @@ static int yaffs2_rd_checkpt_tnodes(struct yaffs_obj *obj)
                if (tn) {
                        ok = (yaffs2_checkpt_rd(dev, tn, dev->tnode_size) ==
                              dev->tnode_size);
                if (tn) {
                        ok = (yaffs2_checkpt_rd(dev, tn, dev->tnode_size) ==
                              dev->tnode_size);

-               } else
+               } else {

                        ok = 0;
                        ok = 0;

+                }

 
                if (tn && ok)
                        ok = yaffs_add_find_tnode_0(dev,
 
                if (tn && ok)
                        ok = yaffs_add_find_tnode_0(dev,


@@ -632,9 +633,9 @@ static int yaffs2_rd_checkpt_objs(struct yaffs_dev *dev)
                    TENDSTR), cp.obj_id, cp.parent_id, cp.variant_type,
                   cp.hdr_chunk));
 
                    TENDSTR), cp.obj_id, cp.parent_id, cp.variant_type,
                   cp.hdr_chunk));
 

-               if (ok && cp.obj_id == ~0)
+               if (ok && cp.obj_id == ~0) {

                        done = 1;
                        done = 1;

-               else if (ok) {
+               } else if (ok) {

                        obj =
                            yaffs_find_or_create_by_number(dev, cp.obj_id,
                                                           cp.variant_type);
                        obj =
                            yaffs_find_or_create_by_number(dev, cp.obj_id,
                                                           cp.variant_type);


@@ -650,8 +651,9 @@ static int yaffs2_rd_checkpt_objs(struct yaffs_dev *dev)
                                            (struct list_head *)hard_list;
                                        hard_list = obj;
                                }
                                            (struct list_head *)hard_list;
                                        hard_list = obj;
                                }

-                       } else
+                       } else {

                                ok = 0;
                                ok = 0;

+                        }

                }
        }
 
                }
        }
 


@@ -918,8 +920,9 @@ int yaffs2_handle_hole(struct yaffs_obj *obj, loff_t new_size)
                        if (written == this_write) {
                                pos += this_write;
                                increase -= this_write;
                        if (written == this_write) {
                                pos += this_write;
                                increase -= this_write;

-                       } else
+                       } else {

                                small_increase_ok = 0;
                                small_increase_ok = 0;

+                        }

                }
 
                yaffs_release_temp_buffer(dev, local_buffer, __LINE__);
                }
 
                yaffs_release_temp_buffer(dev, local_buffer, __LINE__);


@@ -1410,9 +1413,9 @@ int yaffs2_scan_backwards(struct yaffs_dev *dev)
                                                in->yst_mode = oh->yst_mode;
                                                yaffs_load_attribs(in, oh);
                                                in->lazy_loaded = 0;
                                                in->yst_mode = oh->yst_mode;
                                                yaffs_load_attribs(in, oh);
                                                in->lazy_loaded = 0;

-                                       } else
+                                       } else {

                                                in->lazy_loaded = 1;
                                                in->lazy_loaded = 1;

-
+                                        }

                                        in->hdr_chunk = chunk;
 
                                } else if (!in->valid) {
                                        in->hdr_chunk = chunk;
 
                                } else if (!in->valid) {


@@ -1476,8 +1479,7 @@ int yaffs2_scan_backwards(struct yaffs_dev *dev)
                                                               variant.dir_variant.children);
                                        } else if (!parent
                                                   || parent->variant_type !=
                                                               variant.dir_variant.children);
                                        } else if (!parent
                                                   || parent->variant_type !=

-                                                  YAFFS_OBJECT_TYPE_DIRECTORY)
-                                       {
+                                                  YAFFS_OBJECT_TYPE_DIRECTORY) {

                                                /* Hoosterman, another problem....
                                                 * We're trying to use a non-directory as a directory
                                                 */
                                                /* Hoosterman, another problem....
                                                 * We're trying to use a non-directory as a directory
                                                 */