Re: [AD] allegro doesn't seem to be 64 bit clean.

[Philipp Thomas <pthomas@xxxxxxxxxx>]

* Eric Botcazou (ebotcazou@xxxxxxxxxx) [20030131 15:49]:

> Frankly, this would unnecessarily makes the code less readable. Allegro 
> already uses extensively the conversion pointer<-->long so I think we'd 
> better keep consistent in this matter.

OK. But how should the following case from guiproc.c(d_text_list_proc) be
handled?

[d is DIALOG *]

d->dp3 = (void *)((int)d->dp3 + 1);

You get *two* warnings here because a pointer won't fit in an int and 
'void *' is larger than int. I guess, following your reasoning, this should
be changed to:

d->dp3 = (void *)(long int)((long int)d->dp3 + 1);

Right?

Philipp

-- 
Philipp Thomas <pthomas@xxxxxxxxxx>
SuSE Linux AG, Deutschherrnstr. 15-19, D-90429 Nuremberg, Germany