[MPlayer-dev-eng] patch for xinerama

[Attila Kinali]

On Tue, 15 Jan 2002 12:19:15 +1300
daniel carter <hedonist at win.co.nz> wrote:


> Just out of interest ( i'm still learning the ways of C ), what was the
> reason for creating the window and then moving it, rather than just creating
> it with the correct coordinates in the first place as my patch did?  The way
> my patch worked the video drivers didn't need to be xinerama aware, it was
> all done in the initialisation of x11_common.c.  With your method each video
> output driver needs a

It's simple: too many window managers ignore/overide those settings
while mapping a window. But they dont if the window is already mapped.
This was recommended by Mattias Haselmann some months ago as the ICCCM way
(search the archives, it should be his only mail).

A drawback of this system is, that the window is afterwards always
in the right top corner of the selected head as the window placing
system of the WM is overriden. 

If any WM developer is around: why not adding an interface to
tell the WM which head a program likes to be on ?
(this system is just an ugly workaround)
 
> #ifdef HAVE_XINERAMA
>    vo_x11_xinerama_move(mDisplay,mywindow);
> #endif

I know, that's a bad thing, but not avoidable if you have to move
the window after mapping (mapping is done in each vo_ driver individualy).
 
> Also, if you have time, something i was going to work on in my patch was an
> option like
> -xineramascreen all
> to display on all screens.  Would be great for displaying widescreen movies
> across 2 4:3 screens side by side.

Oh, That should be simple to do. IMHO just adding a switch -noxinerama in
combination with -fs (i'll do it over the weekend, not enough time
at the moment).

		Attila Kinali

-- 
I am a moslem, i am a terrorist.