[MPlayer-dev-eng] [PATCH] SDL widescreen modes

[Klokan Petr Pridal]

On Wed, Jul 10, 2002 at 06:47:43AM +0200, Felix Buenemann wrote:
> On Thursday 04 July 2002 14:40, Klokan Petr Pridal wrote:
> > Hi,
> >
> > > > Besides the fact that the code is junk (it has some obvious bugs if I
> > > > saw correctly, eg = vs. ==), what's the use of it?
> > >
> > > Btw. I think my answer sounds a bit rude, wasn't meant to, sorry!
> >
> > There was really this error. I'm ashamed, now I got a lammer stamp :-(. I
> > hope it's not forever. Thanks for your check-up, so the error was not
> > published yet.
> >
> > Repaired patch is included.
> This one looks OK, but I still don't know what's the use of it.
> 
> Can you please explain your setup and give an example of a movie resolution 
> and which mode would be chosen with your code and which mode with the old 
> code.

For example (I remember, because I'm not near the computer with patched version):

Video file:
Standard divx avi encoded from 16:9 video dvd resized to 640x360. 

All available modes on widescreen display on TiBook:
Mode: 0:  1152 x 768
Mode: 1:  1024 x 768
Mode: 2:  896 x 600
Mode: 3:  800 x 600
Mode: 4:  720 x 480
Mode: 5:  640 x 480

With old code this will use 640x480 (There are black bars on widescreen display on all four sides: top, bottom, left, right) 
New patched code will use: 

SET Mode: 4:  720 x 480

So on the display are only small top and bottom black bars. 
If there are no extra widescreen modes available, no change in choosing of modes will happend. 

Klokan
-- 
 (\_/)    Klokan Petr Pridal
 (o o)    ----------------------------
  (.)     student FI MU Brno, webmaster http://www.linux.cz/
 (] [) _  mailto:klokan at seznam.cz
 {~~~}/ ) ICQ:3674202
(')  (')  http://klokan.misto.cz/
 \_)-(_)  Look at http://www.fi.muni.cz/~xpridal/