On Tuesday, January 04, 2011 6:25:02 am Kostik Belousov wrote:
> On Tue, Jan 04, 2011 at 11:40:45AM +0100, Giorgos Keramidas wrote:
> @@ -123,12 +121,28 @@ main(argc, argv)
>               }
>               exit(0);
>       }
> -     exit (1);
> +     exit(1);
> +}
> +
> +static int
> +parseint(const char *str, const char *errname)
> +{
> +     char *endp;
> +     long res;
> +
> +     errno = 0;
> +     res = strtol(str, &endp, 10);
> +     if (errno != 0 || endp == str || *endp != '\0')
> +             err(1, "%s shall be a number", errname);

Small nit, maybe use 'must' instead of 'shall'.

-- 
John Baldwin
_______________________________________________
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"

Reply via email to