I absolutely agree with this patch :)

Thanks,
Guillaume

On Fri, 07 Apr 2006 11:33:57 +0200 "Ulf-D. Ehlert" <[EMAIL PROTECTED]> wrote:

> Variable initialized only in DEBUG mode:
> 
> --- parted-1.6.25.1/libparted/fs_hfs/hfs.c      2005-11-11 13:34:12.000000000 
> +0100
> +++ parted-1.6.25.1/libparted/fs_hfs/hfs.c      2006-04-04 14:30:39.000000000 
> +0200
> @@ -196,7 +196,12 @@
>         PED_ASSERT (fs->geom != NULL, return 0);
>         PED_ASSERT (geom != NULL, return 0);
>         PED_ASSERT (fs->geom->dev == geom->dev, return 0);
> +#ifdef DEBUG
>         PED_ASSERT ((hgee = hfs_get_empty_end(fs)) != 0, return 0);
> +#else
> +       if ((hgee = hfs_get_empty_end(fs)) == 0)
> +               return 0;
> +#endif
> 
>         if (ped_geometry_test_equal(fs->geom, geom))
>                 return 1;
> @@ -915,7 +920,12 @@
>         PED_ASSERT (fs->geom != NULL, return 0);
>         PED_ASSERT (geom != NULL, return 0);
>         PED_ASSERT (fs->geom->dev == geom->dev, return 0);
> +#ifdef DEBUG
>         PED_ASSERT ((hgms = hfsplus_get_min_size (fs)) != 0, return 0);
> +#else
> +       if ((hgms = hfsplus_get_min_size (fs)) == 0)
> +               return 0;
> +#endif
> 
>         if (ped_geometry_test_equal(fs->geom, geom))
>                 return 1;
> 
> _______________________________________________________________
> SMS schreiben mit WEB.DE FreeMail - einfach, schnell und
> kostenguenstig. Jetzt gleich testen! http://f.web.de/?mc=021192
> 
> 
> 
> _______________________________________________
> bug-parted mailing list
> bug-parted@gnu.org
> http://lists.gnu.org/mailman/listinfo/bug-parted


_______________________________________________
bug-parted mailing list
bug-parted@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-parted

Reply via email to