[MPlayer-dev-eng] PMP demuxer for mplayer

[Diego Biurrun]

On Sun, Aug 03, 2008 at 05:57:06PM +0800, wan0eve wrote:
> I wrote a patch to make mplayer support pmp file, a type of media file in  
> PSP(PlayStation Portable).
> 
> I hope it can be add to mplayer.

Maybe you noticed the following text in the sources:

> --- libmpdemux/demuxer.c	（revision 27401）
> +++ libmpdemux/demuxer.c	（working copy）
> @@ -80,6 +80,7 @@
>   extern const demuxer_desc_t demuxer_desc_lavf_preferred;
>   extern const demuxer_desc_t demuxer_desc_aac;
>   extern const demuxer_desc_t demuxer_desc_nut;
> +extern const demuxer_desc_t demuxer_desc_pmp;
> 
>   /* Please do not add any new demuxers here. If you want to implement a new
>    * demuxer, add it to libavformat, except for wrappers around external

here

> @@ -152,6 +153,7 @@
>   #ifdef HAVE_XMMS
>       &demuxer_desc_xmms,
>   #endif
> +    &demuxer_desc_pmp,
>       /* Please do not add any new demuxers here. If you want to implement a new
>        * demuxer, add it to libavformat, except for wrappers around external
>        * libraries and demuxers requiring binary support. */

and here

We mean it, please implement it as a libavformat demuxer in FFmpeg instead.

Should you decide to rewrite this for FFmpeg, please note that your mailer
mangled the patch, next time attach it.  Also, use just spaces, not a
mix of spaces and tabs for indentation.

Do samples of this format exist in our samples collection
(http://samples.mplayerhq.hu)?

Diego