[PATCH] dix/configure: remove null root cursor option

Peter Hutterer peter.hutterer at who-t.net
Thu Jan 14 21:07:08 PST 2010


On Thu, Jan 14, 2010 at 05:18:28PM +0200, Tiago Vignatti wrote:
> 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 <tiago.vignatti at nokia.com>
> ---
>  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

Acked-by: Peter Hutterer <peter.hutterer at who-t.net>

though I'd recommend wait for Daniel or others to comment on whether this is
still needed.

Cheers,
  Peter


More information about the xorg-devel mailing list