# HG changeset patch # User reimar # Date 1316470272 0 # Node ID e7f393cb9d0f1a2d78000d84886df03db31ca4d7 # Parent b48986b929135116f3a545b7b318364d606b2c8f Get rid of X11_FULLSCREEN. It seems likely the code does not compile when it is not defined anyway, it is unclear what its purpose is, and lastly vos changing there behaviour depending on whether some otherwise completely unrelated vo (e.g. vo_gl) is enabled seems like really bad idea. diff -r b48986b92913 -r e7f393cb9d0f libvo/vo_dxr2.c --- a/libvo/vo_dxr2.c Mon Sep 19 22:03:36 2011 +0000 +++ b/libvo/vo_dxr2.c Mon Sep 19 22:11:12 2011 +0000 @@ -805,7 +805,6 @@ { // I'd like to have this done in an x11 independent way // It's because of this that we are limited to vo_x11 for windowed overlay :-( -#ifdef X11_FULLSCREEN if(sub_vo && sub_vo_win) { int e=vo_x11_check_events(mDisplay); if ( !(e&VO_EVENT_RESIZE) && !(e&VO_EVENT_EXPOSE) ) return; @@ -813,7 +812,6 @@ XClearWindow(mDisplay, vo_window); dxr2_set_overlay_window(); } -#endif } static int preinit(const char *arg) { diff -r b48986b92913 -r e7f393cb9d0f libvo/vo_xover.c --- a/libvo/vo_xover.c Mon Sep 19 22:03:36 2011 +0000 +++ b/libvo/vo_xover.c Mon Sep 19 22:11:12 2011 +0000 @@ -107,7 +107,6 @@ drwcX=drwX=vo_dx; drwcY=drwY=vo_dy; drwWidth=vo_dwidth; drwHeight=vo_dheight; } -#if X11_FULLSCREEN if (vo_fs) { aspect(&dwidth,&dheight,A_ZOOM); @@ -120,7 +119,6 @@ mp_msg(MSGT_VO, MSGL_V, "[xvidix-fs] dcx: %d dcy: %d dx: %d dy: %d dw: %d dh: %d\n", drwcX, drwcY, drwX, drwY, drwWidth, drwHeight); } -#endif vo_dwidth=drwWidth; vo_dheight=drwHeight; @@ -274,9 +272,7 @@ { #endif -#ifdef X11_FULLSCREEN if ( ( flags&VOFLAG_FULLSCREEN )||(flags & VOFLAG_SWSCALE) ) aspect(&d_width, &d_height, A_ZOOM); -#endif dwidth = d_width; dheight = d_height; /* Make the window */ diff -r b48986b92913 -r e7f393cb9d0f libvo/vo_xvidix.c --- a/libvo/vo_xvidix.c Mon Sep 19 22:03:36 2011 +0000 +++ b/libvo/vo_xvidix.c Mon Sep 19 22:11:12 2011 +0000 @@ -110,7 +110,6 @@ drwHeight = vo_dheight; } -#if X11_FULLSCREEN if (vo_fs) { aspect(&dwidth, &dheight, A_ZOOM); @@ -129,7 +128,6 @@ "[xvidix-fs] dcx: %d dcy: %d dx: %d dy: %d dw: %d dh: %d\n", drwcX, drwcY, drwX, drwY, drwWidth, drwHeight); } -#endif vo_dwidth = drwWidth; vo_dheight = drwHeight; @@ -247,10 +245,8 @@ } mp_msg(MSGT_VO, MSGL_V, "Using colorkey: %x\n", colorkey); -#ifdef X11_FULLSCREEN if ((flags & VOFLAG_FULLSCREEN) || (flags & VOFLAG_SWSCALE)) aspect(&d_width, &d_height, A_ZOOM); -#endif dwidth = d_width; dheight = d_height; /* Make the window */ diff -r b48986b92913 -r e7f393cb9d0f libvo/x11_common.c --- a/libvo/x11_common.c Mon Sep 19 22:03:36 2011 +0000 +++ b/libvo/x11_common.c Mon Sep 19 22:11:12 2011 +0000 @@ -28,8 +28,6 @@ #include "libavutil/common.h" #include "x11_common.h" -#ifdef X11_FULLSCREEN - #include #include #include @@ -1688,8 +1686,6 @@ } #endif -#endif /* X11_FULLSCREEN */ - /* * Scan the available visuals on this Display/Screen. Try to find diff -r b48986b92913 -r e7f393cb9d0f libvo/x11_common.h --- a/libvo/x11_common.h Mon Sep 19 22:03:36 2011 +0000 +++ b/libvo/x11_common.h Mon Sep 19 22:11:12 2011 +0000 @@ -25,12 +25,6 @@ #include "config.h" -#if defined(CONFIG_GL) || defined(CONFIG_X11) || defined(CONFIG_XV) -#define X11_FULLSCREEN 1 -#endif - -#ifdef X11_FULLSCREEN - #define vo_wm_LAYER 1 #define vo_wm_FULLSCREEN 2 #define vo_wm_STAYS_ON_TOP 4 @@ -90,7 +84,6 @@ void vo_x11_border(void); void vo_x11_ewmh_fullscreen( Window win, int action ); -#endif extern Window vo_window; extern GC vo_gc;