| Re: [AD] Patches for allegro 4.1.9 |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
* Eric Botcazou (ebotcazou@xxxxxxxxxx) [20030205 14:58]:
> --- src/x/xwin.c
> +++ src/x/xwin.c
> @@ -2178,7 +2178,7 @@
> break;
> case ClientMessage:
> /* Window close request */
> - if (event->xclient.data.l[0] == wm_delete_window) {
> + if ((Atom)event->xclient.data.l[0] == wm_delete_window) {
> if (_xwin.close_button_callback)
> _xwin.close_button_callback();
>
> Why to cast to 'Atom'? 'l' is a 'long' according to my docs.
l is a long, but wm_delete_window is XInternAtom, i.e. Atom. I'd rather cast
l to Atom than wm_delete_window to 'long'. But it's your choice to decide.
Philipp
--
Philipp Thomas <pthomas@xxxxxxxxxx>
SuSE Linux AG, Deutschherrnstr. 15-19, D-90429 Nuremberg, Germany