[PATCH] primes(6) -- remove unnecessary code

Steve Kargl sgk at troutmask.apl.washington.edu
Fri Dec 27 18:07:52 UTC 2019


On Fri, Dec 27, 2019 at 09:55:10AM -0800, Steve Kargl wrote:
> This patch removes an unnecessary check as
> the condition can never occur.
> 
> Index: primes.c
> ===================================================================
> --- primes.c	(revision 355983)
> +++ primes.c	(working copy)
> @@ -129,7 +129,7 @@
>  	switch (argc) {
>  	case 2:
>  		/* Start and stop supplied on the command line. */
> -		if (argv[0][0] == '-' || argv[1][0] == '-')
> +		if (argv[1][0] == '-')
>  			errx(1, "negative numbers aren't permitted.");
>  
>  		errno = 0;

Hmm, I withdraw the patch.  One can get a negative start 
value, but one would need to force that condition via
a subversion via getopt.

-- 
Steve


More information about the freebsd-current mailing list