[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Build failure on itanium/IA64
From: |
Arnaud Desitter |
Subject: |
Re: Build failure on itanium/IA64 |
Date: |
Mon, 12 Nov 2001 18:15:37 +0000 (GMT) |
>
> On Fri, Nov 09, 2001 at 23:32:33 +0100, J.H.M. Dassen (Ray) wrote:
> > > #1 Get ddd from the CVS repository on sourceforge.net
> >
> > Debian is slowly moving towards a new release; switching to a CVS version is
> > not an option. Applying patches from CVS selectively is though.
>
> I just checked, and the problem is reproducable with current DDD CVS:
> caballero% c++ -DHAVE_CONFIG_H -I. -I../../ddd -I. -I../../ddd/.. -isystem
> /usr/X11R6/include -DNDEBUG -O2 -O2 -Werror -W -Wall -trigraphs -c
> ../../ddd/ddd.C
> cc1plus: warnings being treated as errors
> ../../ddd/ddd.C: In function `void BlinkCB (void *, XtIntervalId *)':
> ../../ddd/ddd.C:5379: warning: cast to pointer from integer of different size
>
Essentially, a "void*" is used to pass a boolean in a X call-back.
Not beautiful, but it should work.
The "proper" way, i
Replace
blink_timer = XtAppAddTimeOut(XtWidgetToApplicationContext(led_w),
app_data.busy_blink_rate, BlinkCB,
XtPointer(int(!set)));
by
blink_timer = XtAppAddTimeOut(XtWidgetToApplicationContext(led_w),
app_data.busy_blink_rate, BlinkCB,
(!set ? 1 : 0) );
And
in "BlinkCB":
Replace:
bool set = bool(long(client_data));
by
bool set = client_data?true:false;
and,
Replace:
BlinkCB(XtPointer(int(true)), &blink_timer);
by
BlinkCB(XtPointer(true ? 1:0), &blink_timer);
So you can submit a patch if you want. It may be included "at some point".
Cheers,
Arnaud
_____________________________________________________________________
This message has been checked for all known viruses by Star Internet
delivered through the MessageLabs Virus Scanning Service. For further
information visit http://www.star.net.uk/stats.asp or alternatively call
Star Internet for details on the Virus Scanning Service.