[AC21.5] Cleaning out old work #1: double-cast to fix C++ build
"Stephen J. Turnbull" <[email protected]> Mon, 4 Jan 2016 12:44:35 +0900
| Newsgroups | gmane.emacs.xemacs.patches |
|---|---|
| Message-ID | <[email protected]> |
APPROVE COMMIT 21.5 This is a now-ancient patch from Jerry. I haven't actually run with it but I've verified that the build fails without the patch and succeeds with it, so I'm pushing. I wonder if maybe "long" shouldn't be "EMACS_INT"? In theory EMACS_INT could be long long on some platform to accomodate void*, IIRC. diff -U0 -r acf690294825 -r 28813695b141 lwlib/ChangeLog --- a/lwlib/ChangeLog Wed Oct 28 09:17:22 2015 +0100 +++ b/lwlib/ChangeLog Mon Jan 04 12:14:20 2016 +0900 @@ -0,0 +1,5 @@ +2015-01-07 Jerry James <[email protected]> + + * lwlib-Xaw.c (xaw_scrollbar_scroll): Add typecast to fix g++ build, + problem reported by Marcus Harnisch. + diff -r acf690294825 -r 28813695b141 lwlib/lwlib-Xaw.c --- a/lwlib/lwlib-Xaw.c Wed Oct 28 09:17:22 2015 +0100 +++ b/lwlib/lwlib-Xaw.c Mon Jan 04 12:14:20 2016 +0900 @@ -613,10 +613,10 @@ return; id = instance->info->id; - event_data.slider_value = (int) call_data; + event_data.slider_value = (int) (long) call_data; event_data.time = 0; - if ((int) call_data > 0) + if ((int) (long) call_data > 0) /* event_data.action = SCROLLBAR_PAGE_DOWN;*/ event_data.action = SCROLLBAR_LINE_DOWN; else