Bug 4244 - XcursorSetTheme(display, NULL)
Summary: XcursorSetTheme(display, NULL)
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Lib/Xcursor (show other bugs)
Version: unspecified
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Xorg Project Team
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 1690
  Show dependency treegraph
 
Reported: 2005-08-25 12:20 UTC by Owen Taylor
Modified: 2005-10-18 22:27 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
xcursor-4244-reset-theme-name.diff (3.21 KB, patch)
2005-09-12 09:05 UTC, Federico Mena-Quintero
no flags Details | Splinter Review

Description Owen Taylor 2005-08-25 12:20:06 UTC
Would be much more useful if XcursorSetTheme(display, NULL) 
if set the theme to the initial default rather than to NULL.
(Which seems to look things up in the theme 'default', which is
not the initial default value, since that depends on Xresources 
and the environment.)
Comment 1 Keith Packard 2005-08-25 13:43:05 UTC
It would be even nicer if we could figure out how cursor themes should actually
work and move in that direction...

Comment 2 Owen Taylor 2005-08-25 16:06:26 UTC
I think it's working pretty for us at the current time.
Comment 3 Federico Mena-Quintero 2005-09-09 18:38:44 UTC
About the opening comment - what would the "initial default" be?
Comment 4 Federico Mena-Quintero 2005-09-09 18:54:07 UTC
Oh, I think I get it.

So XcursorSetTheme(dpy, NULL) selects the core ugly cursors, but you want it to
pick up the Xcursor.theme xresource?
Comment 5 Federico Mena-Quintero 2005-09-09 18:55:09 UTC
Are the resources for Xcursor listed/documented anywhere?
Comment 6 Federico Mena-Quintero 2005-09-12 09:05:23 UTC
Created attachment 3233 [details] [review]
xcursor-4244-reset-theme-name.diff

This patch fixes the bug for me.  The bug in particular we are hitting is this:


https://bugzilla.novell.com/show_bug.cgi?id=114490
Comment 7 Keith Packard 2005-09-12 09:51:20 UTC
I've committed Federico's patch to xlibs/Xcursor, but not to xorg/lib/Xcursor.
For that, I believe we need to push it through the release queue.
Comment 8 Adam Jackson 2005-10-19 15:27:25 UTC
applied to head, should show up in RC2.  thanks!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.