[MPlayer-dev-eng] Updated nailfps filter patches

[Diego Biurrun]

On Sun, Nov 11, 2007 at 05:27:40AM -0500, Christopher Monty Montgomery wrote:
> Attached are the following patches, seperated apart to make them easy
> to read, but intended to be applied in sequence.  They are against SVN
> of 2007/11/09.

> --- libmpcodecs/vf_nailfps.c	(revision 0)
> +++ libmpcodecs/vf_nailfps.c	(revision 0)
> @@ -0,1 +1,177 @@
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <string.h>

Please add a standard (GPL) license header.

> --- DOCS/man/en/mplayer.1	(revision 25005)
> +++ DOCS/man/en/mplayer.1	(working copy)
> @@ -6054,7 +6054,38 @@
>  .
>  .TP
> +.B nailfps[=hysteresis]
> +Nailfps works like MEncoder's -ofps but is a video filter that also
> +works in MPlayer. It adds or drops frames as needed to enforce the 
> +fixed frame rate specified by -fps, regardless of the actual frame 
> +rate declared by the stream.  This is useful when dumping or transcoding 
> +variable frame rate streams like ASF, MOV, RM or WMV, or damaged streams 
> +that may be missing frames.

New sentences should go on new lines in the manual page.  You need to
escape the dashes with a backslash when they are not used as hyphens,
i.e. \-fps.

> +.IPs "<hysteresis>\ \ "

The quotes and backslash-escaped spaces are unnecessary.

> --- DOCS/man/en/mplayer.1	(revision 25005)
> +++ DOCS/man/en/mplayer.1	(working copy)
> @@ -5225,7 +5225,14 @@
>  .
> +.TP
> +.B "nailsync\ \ \ "

ditto

> --- libaf/af_nailsync.c	(revision 0)
> +++ libaf/af_nailsync.c	(revision 0)
> @@ -0,0 +1,126 @@
> +/* This audio filter tracks video pts and hard-aligns the sample
> +   stream to the video.  Playback to console already implements its
> +   own synchronization mechanisms, but none of those are available to
> +   non-console playback, such as a/v dumping.
> +*/
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <string.h>
> +#include <inttypes.h>

license header please

Diego