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 <[email protected]>
---
 configure.ac            |    7 -------
 dix/cursor.c            |   17 -----------------
 include/dix-config.h.in |    3 ---
 3 files changed, 0 insertions(+), 27 deletions(-)

diff --git a/configure.ac b/configure.ac
index 43bc365..3692033 100644
--- a/configure.ac
+++ b/configure.ac
@@ -565,9 +565,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])
@@ -1146,10 +1143,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 6103b15..9f81ded 100644
--- a/dix/cursor.c
+++ b/dix/cursor.c
@@ -520,26 +520,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);
@@ -553,7 +537,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 f46260d..b8a735e 100644
--- a/include/dix-config.h.in
+++ b/include/dix-config.h.in
@@ -405,9 +405,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
 
-- 
1.6.0.4

_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to