From 3eee14d4e769ad063e71ccb476f722188a66d50f Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 22 Aug 2023 11:18:00 +0200 Subject: [PATCH] tdf#156825 Half of the splash screen is visible on both monitors under Linux Blind fix, since I don't have two screens. regression from commit 4c00a5bce384c5fa3b46cf5bb29c7e9b1e506ea0 Author: Noel Grandin Date: Thu Jul 20 11:27:17 2023 +0200 Assume that, if we have X11, it supports Xinerama It looks like I missed the removal of some USE_XINERAMA pieces Change-Id: I402db170916efe935d8ff9f96cc9b327070809b7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155939 Tested-by: Jenkins Reviewed-by: Noel Grandin --- desktop/Executable_oosplash.mk | 6 ------ desktop/unx/source/splashx.c | 8 +------- solenv/gbuild/PrecompiledHeaders.mk | 1 - vcl/Library_vclplug_gen.mk | 3 --- 4 files changed, 1 insertion(+), 17 deletions(-) diff --git a/desktop/Executable_oosplash.mk b/desktop/Executable_oosplash.mk index 308c76945aef..2808e0113510 100644 --- a/desktop/Executable_oosplash.mk +++ b/desktop/Executable_oosplash.mk @@ -24,15 +24,9 @@ $(eval $(call gb_Executable_add_cobjects,oosplash,\ )) ifneq ($(USING_X11),) - -$(eval $(call gb_Executable_add_defs,oosplash,\ - -DUSE_XINERAMA \ -)) - $(eval $(call gb_Executable_add_libs,oosplash,\ -lXinerama \ )) - endif ifneq ($(DISABLE_GUI),TRUE) diff --git a/desktop/unx/source/splashx.c b/desktop/unx/source/splashx.c index 4f7239ab6159..4dc50a740695 100644 --- a/desktop/unx/source/splashx.c +++ b/desktop/unx/source/splashx.c @@ -16,9 +16,7 @@ #include #include -#ifdef USING_X11 #include -#endif #include #include @@ -406,10 +404,8 @@ static int splash_init_display( struct splash* splash, int argc, char** argv ) { char *display_name = NULL; int i; -#ifdef USING_X11 int n_xinerama_screens = 1; XineramaScreenInfo* p_screens = NULL; -#endif for ( i = 0; i < argc; i++ ) { @@ -442,7 +438,6 @@ static int splash_init_display( struct splash* splash, int argc, char** argv ) splash->display_x_pos = 0; splash->display_y_pos = 0; -#ifdef USING_X11 p_screens = XineramaQueryScreens( splash->display, &n_xinerama_screens ); if( p_screens ) { @@ -459,7 +454,6 @@ static int splash_init_display( struct splash* splash, int argc, char** argv ) } XFree( p_screens ); } -#endif return 1; } @@ -812,6 +806,6 @@ struct splash* splash_create(rtl_uString* pAppPath, int argc, char** argv) } -#endif // ENABLE_QUICKSTART_LIBPNG +#endif // defined(ENABLE_QUICKSTART_LIBPNG) && HAVE_FEATURE_UI /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/solenv/gbuild/PrecompiledHeaders.mk b/solenv/gbuild/PrecompiledHeaders.mk index d7b84e991578..60445c28efbe 100644 --- a/solenv/gbuild/PrecompiledHeaders.mk +++ b/solenv/gbuild/PrecompiledHeaders.mk @@ -149,7 +149,6 @@ gb_PrecompiledHeader_ignore_flags_system := \ -DSYSTEM_NSS \ -DHAVE_VALGRIND_HEADERS \ -DUSE_RANDR \ --DUSE_XINERAMA_XORG \ -DDISABLE_CVE_TESTS \ -DCPPUNIT_PLUGIN_EXPORT='extern "C" SAL_DLLPUBLIC_EXPORT' \ -DOOO_DLLIMPLEMENTATION_TEST \ diff --git a/vcl/Library_vclplug_gen.mk b/vcl/Library_vclplug_gen.mk index dea289a8e0c5..e0086e6c4dcb 100644 --- a/vcl/Library_vclplug_gen.mk +++ b/vcl/Library_vclplug_gen.mk @@ -134,9 +134,6 @@ endif ## handle Xinerama ifneq ($(USING_X11),) -$(eval $(call gb_Library_add_defs,vclplug_gen,\ - -DUSE_XINERAMA_XORG \ -)) ifeq ($(XINERAMA_LINK),dynamic) $(eval $(call gb_Library_add_libs,vclplug_gen,\ -lXinerama \