Hi folks,
This fixes some 64bit problems in Xt calls with va_lists.
XtVaSetValues is a list terminated with a true NULL pointer.
0 is not sufficient, because it is just 32bit wide on 64bit systems.
XtVaSetValues needs to be terminated with a true NULL pointer too,
while XtArgVal is asserted to be long enough, a (char*)NULL is better
there.
Ciao, Marcus
--- xemacs-21.5.18/lwlib/xlwgauge.c.xx 2005-02-28 11:15:26.000000000 +0100
+++ xemacs-21.5.18/lwlib/xlwgauge.c 2005-02-28 11:15:34.000000000 +0100
@@ -838,7 +838,7 @@
if(( gw->gauge.autoScaleUp && (int) value > gw->gauge.v1) ||
(gw->gauge.autoScaleDown && (int) value < gw->gauge.v1/3 ))
{
- XtVaSetValues(w, XtNvalue, value, 0) ;
+ XtVaSetValues(w, XtNvalue, value, NULL) ;
return ;
}
--- xemacs-21.5.18/src/frame-x.c.xx 2005-02-28 11:16:00.000000000 +0100
+++ xemacs-21.5.18/src/frame-x.c 2005-02-28 11:18:18.899716466 +0100
@@ -758,12 +758,12 @@
Qctext);
XtVaSetValues (w, XtVaTypedArg, extprop,
XtRString, extval, extvallen + 1,
- (XtArgVal) NULL);
+ (char*) NULL);
}
else
XtVaSetValues (w, XtVaTypedArg, extprop, XtRInt,
XINT (val), sizeof (int),
- (XtArgVal) NULL);
+ (char*) NULL);
}
else if (SYMBOLP (prop))
{
@@ -855,7 +855,7 @@
XtRString,
XSTRING_DATA (val),
XSTRING_LENGTH (val) + 1,
- (XtArgVal) NULL);
+ (char*) NULL);
}
#ifdef HAVE_SCROLLBARS
Show replies by date