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 <[email protected]>
> 
> --- 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to