[MPlayer-dev-eng] Updated nailfps filter patches

Rich Felker dalias at aerifal.cx
Mon Nov 12 08:50:22 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.

Lots of cosmetic problems due to using an editor with tab-width!=8.
These need to be fixed before commit.

Also please break the patch up into the following parts:
(1) Prerequisites for nailfps in existing MPlayer code
(2) The filters themselves
(3) ao_pcm changes which are unrelated

Glad you got this working. Hopefully it will be ready for commit in
the near future!

Rich



More information about the MPlayer-dev-eng mailing list