[Pkg-sdl-commits] [libsdl1.2] 02/02: Enable X11 backingstore only when SDL_VIDEO_X11_BACKINGSTORE is set to prevent tearing.

[Felix Geyer]

This is an automated email from the git hooks/post-receive script.

fgeyer pushed a commit to branch master
in repository libsdl1.2.

commit c20a281c1e70e003221af93643644978a203745b
Author: Felix Geyer <fgeyer at debian.org>
Date:   Mon Jun 30 19:08:46 2014 +0200

    Enable X11 backingstore only when SDL_VIDEO_X11_BACKINGSTORE is set to prevent tearing.
    
    * Enable X11 backingstore only when SDL_VIDEO_X11_BACKINGSTORE is set to
      prevent tearing. (Closes: #747168)
      - Add sdl-check-for-SDL_VIDEO_X11_BACKINGSTORE.patch
---
 debian/changelog                                   |  5 +++++
 .../sdl-check-for-SDL_VIDEO_X11_BACKINGSTORE.patch | 24 ++++++++++++++++++++++
 debian/patches/series                              |  1 +
 3 files changed, 30 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 54a242d..3a78cd2 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -3,6 +3,11 @@ libsdl1.2 (1.2.15-10) UNRELEASED; urgency=medium
   [ Hector Oron ]
   * d/control: remove libts-dev dependency. (Closes: #751765)
 
+  [ Felix Geyer ]
+  * Enable X11 backingstore only when SDL_VIDEO_X11_BACKINGSTORE is set to
+    prevent tearing. (Closes: #747168)
+    - Add sdl-check-for-SDL_VIDEO_X11_BACKINGSTORE.patch
+
  -- Felix Geyer <fgeyer at debian.org>  Mon, 30 Jun 2014 19:01:12 +0200
 
 libsdl1.2 (1.2.15-9) unstable; urgency=medium
diff --git a/debian/patches/sdl-check-for-SDL_VIDEO_X11_BACKINGSTORE.patch b/debian/patches/sdl-check-for-SDL_VIDEO_X11_BACKINGSTORE.patch
new file mode 100644
index 0000000..c29811d
--- /dev/null
+++ b/debian/patches/sdl-check-for-SDL_VIDEO_X11_BACKINGSTORE.patch
@@ -0,0 +1,24 @@
+Description: Do not harness backing store by default
+ xorg-server 1.15 enables backing store if composite extension is enabled
+ (default settings). Harnessing backing store through compositor leads to
+ tearing effect.
+ This patch reverts default harnessing backing store to conditional use if
+ SDL_VIDEO_X11_BACKINGSTORE environment variable exists.
+Origin: https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/1280665/comments/1
+Bug: https://bugzilla.libsdl.org/show_bug.cgi?id=2383
+Bug-Debian: https://bugs.debian.org/747168
+
+--- a/src/video/x11/SDL_x11video.c
++++ b/src/video/x11/SDL_x11video.c
+@@ -1088,10 +1088,8 @@
+ 		}
+ 	}
+ 
+-#if 0 /* This is an experiment - are the graphics faster now? - nope. */
+ 	if ( SDL_getenv("SDL_VIDEO_X11_BACKINGSTORE") )
+-#endif
+-	/* Cache the window in the server, when possible */
++	/* Cache the window in the server when possible, on request */
+ 	{
+ 		Screen *xscreen;
+ 		XSetWindowAttributes a;
diff --git a/debian/patches/series b/debian/patches/series
index 91bc117..b45befa 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -3,3 +3,4 @@ dont_propagate_lpthread.diff
 fix_build_joystick_freebsd.diff
 fix_window_resizing.diff
 fix_joystick_misc_axes.diff
+sdl-check-for-SDL_VIDEO_X11_BACKINGSTORE.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-sdl/packages/libsdl1.2.git