From 60b6477f6dc005a1b052be8c5e5a549550a4aa88 Mon Sep 17 00:00:00 2001 From: Tiago Vignatti Date: Thu, 14 Jan 2010 17:18:28 +0200 Subject: [PATCH] dix/configure: remove null root cursor option The default behavior of the server is to start with an invisible root cursor. Be such cursor invisible or inexistent (null), in the end it doesn't matter - for the user. The content on screen will be the same. Besides, there's no difference, in terms of performance, whether such cursor is invisible or simply null. The paths that both take inside the server are roughly the same. Therefore create a null root cursor becomes irrelevant. Signed-off-by: Tiago Vignatti Acked-by: Peter Hutterer Acked-by: Daniel Stone Signed-off-by: Keith Packard --- configure.ac | 7 ------- dix/cursor.c | 17 ----------------- include/dix-config.h.in | 3 --- 3 files changed, 27 deletions(-) diff --git a/configure.ac b/configure.ac index 425db40f6..ec9442bd3 100644 --- a/configure.ac +++ b/configure.ac @@ -572,9 +572,6 @@ AC_ARG_ENABLE(install-libxf86config, [Install libxf86config (default: disabled)]), [INSTALL_LIBXF86CONFIG=$enableval], [INSTALL_LIBXF86CONFIG=no]) -AC_ARG_ENABLE(null-root-cursor, AS_HELP_STRING([--enable-null-root-cursor], [Use an empty root cursor (default: use core cursor)]), - [NULL_ROOT_CURSOR=$enableval], - [NULL_ROOT_CURSOR=no]) AC_ARG_ENABLE(visibility, AC_HELP_STRING([--enable-visibility], [Enable symbol visibility (default: auto)]), [SYMBOL_VISIBILITY=$enableval], [SYMBOL_VISIBILITY=auto]) @@ -1187,10 +1184,6 @@ AC_CHECK_FUNC(strncasecmp, [], AC_DEFINE([NEED_STRNCASECMP], 1, AC_CHECK_FUNC(strcasestr, [], AC_DEFINE([NEED_STRCASESTR], 1, [Do not have 'strcasestr'.])) -if test "x$NULL_ROOT_CURSOR" = xyes; then - AC_DEFINE(NULL_ROOT_CURSOR, 1, [Use an empty root cursor]) -fi - PKG_CHECK_MODULES([XDMCP], [xdmcp], [have_libxdmcp="yes"], [have_libxdmcp="no"]) if test "x$have_libxdmcp" = xyes; then AC_CHECK_LIB(Xdmcp, XdmcpWrap, [have_xdmcpwrap="yes"], [have_xdmcpwrap="no"], [$XDMCP_LIBS]) diff --git a/dix/cursor.c b/dix/cursor.c index 324d73345..5d9c8b080 100644 --- a/dix/cursor.c +++ b/dix/cursor.c @@ -487,26 +487,10 @@ CursorPtr CreateRootCursor(char *unused1, unsigned int unused2) { CursorPtr curs; -#ifdef NULL_ROOT_CURSOR - CursorMetricRec cm; -#else FontPtr cursorfont; int err; XID fontID; -#endif -#ifdef NULL_ROOT_CURSOR - cm.width = 0; - cm.height = 0; - cm.xhot = 0; - cm.yhot = 0; - - AllocARGBCursor(NULL, NULL, NULL, &cm, 0, 0, 0, 0, 0, 0, - &curs, serverClient, (XID)0); - - if (curs == NullCursor) - return NullCursor; -#else fontID = FakeClientID(0); err = OpenFont(serverClient, fontID, FontLoadAll | FontOpenSync, (unsigned)strlen(defaultCursorFont), defaultCursorFont); @@ -520,7 +504,6 @@ CreateRootCursor(char *unused1, unsigned int unused2) if (AllocGlyphCursor(fontID, 0, fontID, 1, 0, 0, 0, ~0, ~0, ~0, &curs, serverClient, (XID)0) != Success) return NullCursor; -#endif if (!AddResource(FakeClientID(0), RT_CURSOR, (pointer)curs)) return NullCursor; diff --git a/include/dix-config.h.in b/include/dix-config.h.in index 84c78db54..e942dacf7 100644 --- a/include/dix-config.h.in +++ b/include/dix-config.h.in @@ -414,9 +414,6 @@ /* Use only built-in fonts */ #undef BUILTIN_FONTS -/* Use an empty root cursor */ -#undef NULL_ROOT_CURSOR - /* Have a monotonic clock from clock_gettime() */ #undef MONOTONIC_CLOCK