Bug 11132

Summary: Xming, Cygwin/X: Tidy warnings, headers and unused variables
Product: xorg Reporter: Colin Harrison <colin.harrison>
Component: Server/DDX/XwinAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium CC: jon.turney
Version: unspecified   
Hardware: Other   
OS: Windows (All)   
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 18093    
Attachments:
Description Flags
Patch to fix
none
Patch to consolidate definition of MOUSE_POLLING_INTERVAL and reduce value none

Description Colin Harrison 2007-06-03 15:50:55 UTC
Tidy warnings.
Tidy some '#include' header file preprocessor directives.
Remove some unused variables (some defined via macros!).
Reduce mouse polling interval to 10 mS.
Comment 1 Colin Harrison 2007-06-03 15:51:48 UTC
Created attachment 10169 [details] [review]
Patch to fix
Comment 2 Jon Turney 2008-12-18 04:44:30 UTC
Applied as commit b0ed1567c1782041cf3d14ec6dd4d622915c7d0c. Thanks.

I dropped the part of the patch which changes the mouse polling interval, because I don't understand it.
Comment 3 Colin Harrison 2008-12-18 09:30:21 UTC
Wiz the cursor around an xeyes in -multiwindow mode, and you will what this timing does effect.
Comment 4 Jon Turney 2009-01-07 15:18:02 UTC
Created attachment 21771 [details] [review]
Patch to consolidate definition of MOUSE_POLLING_INTERVAL and reduce value

Indeed, I see.  How about the attached patch?
Comment 5 Colin Harrison 2009-01-11 12:44:08 UTC
That should do it. One definition is always best :)
Comment 6 Colin Harrison 2009-01-12 03:14:51 UTC
PS Works for me. OK to commit. Thanks.
Comment 7 Jon Turney 2009-01-15 13:59:23 UTC
Commited as 6f065a26db62aaf1d7aab9f9a3d0e6cd09d1c1dd

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.