Re: [patch] remove dead code

[Ingo Schwarze <schwarze@usta.de>]

Hi Ulrich,

Ulrich Spörlein wrote on Fri, May 17, 2013 at 03:00:50AM +0200:

> please see the following patch which removes a bit of dead code.
> With sz being invariant 0, the loop will never do anything.

Right, committed to both bsd.lv and openbsd.org.

> This is Coverity Scan CID 975717

Strangly enough, it seems that Coverity missed that **params is
unused as well.  Anyway, i removed **params, too.

Thanks,
  Ingo

> Index: tree.c
> ===================================================================
> RCS file: /usr/vhosts/mdocml.bsd.lv/cvs/mdocml/tree.c,v
> retrieving revision 1.47
> diff -u -p -r1.47 tree.c
> --- tree.c	18 Sep 2011 14:14:15 -0000	1.47
> +++ tree.c	17 May 2013 00:42:33 -0000
> @@ -58,12 +58,12 @@ print_mdoc(const struct mdoc_node *n, in
>  {
>  	const char	 *p, *t;
>  	int		  i, j;
> -	size_t		  argc, sz;
> +	size_t		  argc;
>  	char		**params;
>  	struct mdoc_argv *argv;
>  
>  	argv = NULL;
> -	argc = sz = 0;
> +	argc = 0;
>  	params = NULL;
>  	t = p = NULL;
>  
> @@ -162,9 +162,6 @@ print_mdoc(const struct mdoc_node *n, in
>  				printf(" ]");
>  		}
>  		
> -		for (i = 0; i < (int)sz; i++)
> -			printf(" [%s]", params[i]);
> -
>  		printf(" %d:%d\n", n->line, n->pos);
>  	}
>  
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv