[MPlayer-cvslog] r27849 - trunk/libvo/x11_common.c

[Reimar Döffinger]

On Wed, Oct 29, 2008 at 11:03:39PM +0100, ben wrote:
> Log:
> Remove X11 backing store: this is now a useless flag.

Sorry, but past X.org versions do not magically change, so how can it be
now useless?

> Also, it is mandatory for Xserver 1.5.x (part of Xorg 7.4, shipped on all
> Linux distributions starting from Oct. 08) and will be removed
> from Xserver 1.6 anyhow ...

Removed meaning? An API change?

> Patch by Stephane Marchesin (marchesin at icps dot u dash strasbg dot fr).
> 
> For more info, see long flame thread at:
> http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2008-August/058323.html

Well, and still nobody has clearly explained the why?!
So, why did you commit it? Just because a patch was sent?
Or to work around buggy/slow/stupid video drivers? Or why?
Btw. I am the maintainer of that code and I still object to changing
code when the arguments for it are all full of nonsense and/or
inconsistent.

Greetings,
Reimar Döffinger