Re: Build failure on itanium/IA64

[Arnaud Desitter]

> 
> 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.