[MPlayer-dev-eng] Updated nailfps filter patches
Diego Biurrun
diego at biurrun.de
Mon Nov 12 09:31:14 CET 2007
On Mon, Nov 12, 2007 at 02:28:03AM -0500, xiphmont at xiph.org wrote:
> Updated version of the patch including requested changes. manpage
> tidied, exit()s removed, messages now use mp_msg.
>
> The are two minor additional changes, both to ao_pcm. One adds a
> 'pipe' option to play more nicely with outputting to a pipe, the
> second fixes a bug that would pop up on bigendian (the variable that
> tracks the file length was clobbered to reverse endianness in place to
> save out in the wave header; that became an issue when used with
> multiple input files).
>
> All in one patch this time as editing it into multiple patches caused
> heartache last time.
Split patches are preferred...
> --- libmpcodecs/vf_nailfps.c (revision 0)
> +++ libmpcodecs/vf_nailfps.c (revision 0)
> @@ -0,0 +1,205 @@
> +/*
> + *
> + * vf_nailfps, part of MPlayer
> + *
> + * Copyright (C) 2007 Monty <monty at xiph.org>
> + *
> + */
> +
> +/* nailfps creates a fixed-frame-rate output, much like -ofps for mencoder. */
Stylistic nit: I prefer to have this description as the first line of
the file header.
> --- DOCS/man/en/mplayer.1 (revision 25005)
> +++ DOCS/man/en/mplayer.1 (working copy)
> @@ -5225,6 +5230,13 @@
> .
> +discontinuities, seeks, and file transitions (when used with \-fixed-vo).
You need to escape all dashes, not just the first one. All unescaped
dashes get interpreted as hyphens.
> @@ -6054,6 +6066,44 @@
> .
> +\-fps is required. The "correct" frame rate to use with \-fps may be
> +different from the stream's claimed frame rate.
> +An "incorrect" rate adds or subtracts single frames from the stream
> +at regular intervals. A "correct" rate only adds/removes frames in short
> +bursts, usually at edit points, scene changes, dropouts or other
> +discontinuities.
New sentences on new lines please.
> +Also note that various other options (eg, \-mc or \-correct-pts) that
e.g.\&
> +Valid range is 0. through 2., default is .1.
This is unclear. Is the range 0.0 - 2.0 with a default of 0.1? Then
say so.
> --- libaf/af_nailsync.c (revision 0)
> +++ libaf/af_nailsync.c (revision 0)
> @@ -0,0 +1,137 @@
> +/*
> + *
> + * af_nailsync, part of MPlayer
> + *
> + */
> +
> +
> +/* Nailsync watches audio pts and alters the metronome in vf_nailfps
> + to fix audio and video sync together */
ditto
Diego
More information about the MPlayer-dev-eng
mailing list