[PATCH] result parameter for ScreenWakeupHandlerProcPtr should be unsigned long

Kevin Brace kevinbrace at gmx.com
Thu Dec 13 20:47:01 UTC 2018


The result parameter defined inside scrnintstr.h for
ScreenWakeupHandlerProcPtr should be unsigned long type not int type.
This led to wrong pointer type compilation warnings when compiling S3
Savage DDX Version 2.3.9 for ABI_VIDEODRV_VERSION >= 23.

Signed-off-by: Kevin Brace <kevinbrace at gmx.com>
---
 include/scrnintstr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/scrnintstr.h b/include/scrnintstr.h
index 9b663fa..111cab5 100644
--- a/include/scrnintstr.h
+++ b/include/scrnintstr.h
@@ -278,7 +278,7 @@ typedef void (*ScreenBlockHandlerProcPtr) (ScreenPtr pScreen,
  * > 0 - activity
  */
 typedef void (*ScreenWakeupHandlerProcPtr) (ScreenPtr pScreen,
-                                            int result);
+                                            unsigned long result);
 
 typedef Bool (*CreateScreenResourcesProcPtr) (ScreenPtr /*pScreen */ );
 
-- 
2.7.4



More information about the xorg-devel mailing list