On Fri, 2009-03-27 at 13:33 +0300, Dan Carpenter wrote: > Smatch (http://repo.or.cz/w/smatch.git/) complains about this unreachable > code so I moved the break; up a line. Compile tested. > > regards, > dan carpenter > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > > --- orig/fs/btrfs/ctree.c 2009-03-27 07:31:42.000000000 +0300 > +++ devel/fs/btrfs/ctree.c 2009-03-27 07:32:26.000000000 +0300 > @@ -3318,8 +3318,8 @@ > for (i = 0; i < nr; i++) { > if (total_size + data_size[i] + sizeof(struct btrfs_item) > > BTRFS_LEAF_DATA_SIZE(root)) { > - break; > nr = i; > + break; > } Whoops, that's not good. This whole function is actually unused after the changes in the experimental tree. -chris -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
