On Mon, Jul 23, 2007 at 10:06:07AM -0600, [EMAIL PROTECTED] wrote:
> +static void parse_larsize(char *str)
> +{
> +     char *p = NULL;
> +     unsigned int size = strtoul(str, &p, 0);

larsize is declared a long while parse_larsize() now uses strtoul() -
they should match.


>               case 's':
> -                     larsize = strtol(optarg, (char **)NULL, 10);
> +                     parse_larsize(optarg);

Another issue was already discussed;

Either make parse_larsize() return a value, or put the code in the
case.

File global larsize is yucky and hurts reuse.


//Peter

-- 
linuxbios mailing list
[email protected]
http://www.linuxbios.org/mailman/listinfo/linuxbios

Reply via email to