[MPlayer-dev-eng] [PATCH] parse_es.c: remove unused parameter

Nico Sabbi nicola_sabbi at fastwebnet.it
Sat Jun 2 19:03:30 CEST 2007


Zuxy Meng wrote:
> Hi,
> 
> 'skipped' isn't unused in the format string so isn't necessary.
> 
> 
> ------------------------------------------------------------------------
> 
> Index: libmpdemux/parse_es.c
> ===================================================================
> --- libmpdemux/parse_es.c	£¨°æ±¾ 23450£©
> +++ libmpdemux/parse_es.c	£¨¹¤×÷¸±±¾£©
> @@ -26,7 +26,7 @@
>    int skipped=0;
>      if (!demux_pattern_3(ds, NULL, MAX_SYNCLEN, &skipped, 0x100)) {
>        if (skipped == MAX_SYNCLEN)
> -      mp_msg(MSGT_DEMUXER, MSGL_ERR, "parse_es: could not sync video stream!\n", skipped);
> +      mp_msg(MSGT_DEMUXER, MSGL_ERR, "parse_es: could not sync video stream!\n");
>        goto eof_out;
>      }
>      next_nal = demux_getc(ds);
> 
> 

trivial patches like this don't need authorization




More information about the MPlayer-dev-eng mailing list