January 2011 - MPlayer-dev-eng - lists.mplayerhq.hu

FFmpeg, svn:externals and where to go from here
by Diego Biurrun 16 Feb '11

16 Feb '11

The FFmpeg Subversion repository has been set readonly, we will not be receiving any more updates from it. The time has come to rethink the MPlayer project structure and the way we handle external libraries that we pull into our source tree. I'm undecided what to do with FFmpeg. I'd personally move towards eliminating embedded library copies, but not everybody agrees. I will rev my patches for running staged configures and reusing the FFmpeg build system more instead of duplicating it badly when I find the time. Then there is libdvdread and libdvdnav. There are many shared devs, but the project is not under our direct control and we don't know if there will be a git migration or not. libdvdread/libdvdnav sees orders of magnitude less changes than FFmpeg. IIRC (but please don't take my word for it and doublecheck) current versions are widely available in distributions now. I would suggest just dropping the svn:externals and using system versions of these libraries. The value added by the embedded copies is small, but the complication is considerable due to the ties to infrastructure of projects that are separate from MPlayer. Diego

9 35

[PATCH] Fractional part of time stamp in OSD
by Christian 10 Feb '11

10 Feb '11

Hi, currently, the OSD shows only the integral part of the time stamp. Attached patch allows to optionally also show the fractional part of the time stamp in the OSD. Kind regards, Christian P.S.: This is useful for example when trying to create EDLs. There, when trying to place a cut-mark just before a scene change, it typically occurs that one notices the scene change just after it had occurred (i.e.: one frame too late). Hence, one has to rewind the stream and go to the frame just before the scene-change. Doing so requires to constantly have an eye on the terminal (for the subsecond time stamp) and one on the movie itself (to assert the scene change did not yet occur). Allowing to show the fractional part of the time stamp in the OSD solves this issue. P.P.S.: The proposed changes of the same approach in 2007 [1] have been included in the attached patch. Another previous attempt in this direction has been ignored in 2008 [2]. [1] http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2007-April/051032.html [2] http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2008-April/056972.html

3 7

[PATCH] EOSD: dump images coordinates in verbose mode
by Nicolas George 09 Feb '11

09 Feb '11

Hi. The attached patch makes EOSD dump the coordinates and color of each image element when running in very verbose mode (-v -v -v). A possible use for that is to compute a bounding box for subtitles. Regards, -- Nicolas George

3 6

[PATCH] Avoid raising error when trying to load a vobsub.
by Clément Bœsch 05 Feb '11

05 Feb '11

Hi, A small patch to avoid the message "SUB: Could not determine file format" each time a vobsub is loaded. The MPEG header is just checked a single time (first line/packet) and after every other format detection failed. -- Clément B.

2 8

3 incoming patches for subtitles
by Clément Bœsch 05 Feb '11

05 Feb '11

Hi, This is 3 subtitles patches. The first one will be needed for the two following ones. Second patch's first goal is to avoid a heap overflow with tmpresult, and also make path cleaner (see details on extended comment). About the last one, it's a known one, but including the vobsub loading. Regards,

2 16

[PATCH] Enhanced Video Renderer (EVR)
by Georgi Petrov 05 Feb '11

05 Feb '11

Hi, I'm trying with the first patch, as simple as possible. Please take a look when you have time and point the things that may require changes. I place temporary the header evr.h in /libvo. As I explained, once it is good enough, I will try to submit it to mingw. Meanwhile I'm going to improve it inside the mingw-w64 project. Regularly I will update it here as well. The second file is vo_evr.c. Right now it doesn't contain anything useful, besides empty functions. The third modified file is the configure script. I think I got it right, but the final word is yours. One final question - should I call vo_evr.c actually vo_dxva.c? I mean - would it be more appropriate? After all the people won't need to know what EVR is, this will want DxVA. From other perspective, DxVA will be available sometime in vo_directx.c as well (DxVA 1) and it is getting somewhat confusing. Greetings, Georgi

4 35

[PATCH] smil params support is added.
by Andrei Katsuk 04 Feb '11

04 Feb '11

Hi, Currently, when parsing a SMIL playlist file, MPlayer is support only <media>, <video> <audio> src tags. Attached patch adds support of param tag like: <video src="test.avi" > <param name="additional-param1" value="additional-value1"/> <param name="additional-param2" value="additional-value2"/> </video> Also paramGroup support is added: <paramGroup xml:id="testgroup"> <param name="additional-param1" value="additional-value1"/> <param name="additional-param2" value="additional-value2"/> </paramGroup> </head> <body> <ref src="test.avi" paramGroup="tesgroup"> Andrei

6 11

[PATCH] Add prefix to filename of files created by PNG video output
by Marco Aurélio Graciotto Silva 02 Feb '11

02 Feb '11

I was wondering if Mplayer couldn't add a prefix to the filename of the PNG created using PNG 'video' output. Thus, I created a small patch that allows the usage of a prefix for the PNG output filename. The new option is "outfile", which is used to define the 'png_outfile_prefix' variable. The default behaviour is the same as Mplayer currently has (default value for 'png_outfile_prefix' is an empty string), so backwards compatibility is kept. The patch was created against MPlayer svn #32707. Regards, Marco Aurélio

3 5

[homepage]: r3558
by Ingo Brückl 31 Jan '11

31 Jan '11

> - <a href="../MPlayer/skins/native_red.tar.gz">HTTP</a> > + <a href="../MPlayer/skins/nativeRed-1.1.tar.bz2">HTTP</a> Someone with server access please delete native_red.tar.gz Ingo

2 1

[PATCH] Improve the readability of dynamic labels which scroll
by Ingo Brückl 31 Jan '11

31 Jan '11

If nobody objects, I'd like to commit this patch. (BTW, the diff of gui/app.h shows the disadvantage of pretty-printing. Although only a few lines added, the diff is lengthy and hard to read.) Explanation: If the text of a dynamic label to be displayed is wider than the given length, it will be scrolled. Currently such a label starts scrolling immediately after it is placed and - even more unpleasant - the start of the text is randomly somewhere within the specified space of the label. Both makes it hard to track and to read. Now such a dynamic label starts left-aligned and begins scrolling through the specified space only after a short delay (2.5 seconds). Every time the start of the text nears the left margin again during the scrolling process it will stop and everything starts all over again, i.e. scrolling after a short delay. Ingo

4 5