dtprintinfo: Use 64-bit values for XtVaGetValues() pointers
authorMarcin Cieslak <saper@saper.info>
Mon, 24 Sep 2012 01:15:03 +0000 (03:15 +0200)
committerJon Trulson <jon@radscan.com>
Mon, 24 Sep 2012 01:43:11 +0000 (19:43 -0600)
XtArgVal should be a type that encompasses XtPointer
and long integer types. In the X.org implementation
it is currently defined as (long).

Don't use (unsigned int *) instead of (Window *).

cde/programs/dtprintinfo/UI/DtDetailsLabel.C
cde/programs/dtprintinfo/libUI/MotifUI/DtDND.C
cde/programs/dtprintinfo/libUI/MotifUI/Icon.c

index d830c677783515a26572699c85f3038daf68905d..a61d45b83f8e14597cb7fc12e6c8c7179402b95b 100644 (file)
@@ -55,9 +55,11 @@ DtDetailsLabel::DtDetailsLabel(MotifUI *parent)
               MESSAGE(JobNumberL), MESSAGE(TimeL), MESSAGE(DateL));
       bottom_label[1] = StringCreate(label);
 
+      XtArgVal arg;
       Dimension highlight;
       XtVaGetValues(parent->BaseWidget(),
-                   XmNhighlightThickness, &highlight, NULL);
+                   XmNhighlightThickness, &arg, NULL);
+      highlight = (Dimension)arg;
       int height = 21 + 2 * highlight;
       blank_pixmap = XCreatePixmap(display, root, 1, height, depth);
       blank_mask = XCreatePixmap(display, root, 1, height, 1);
@@ -73,7 +75,7 @@ DtDetailsLabel::DtDetailsLabel(MotifUI *parent)
    XmString top_string = StringCreate(MESSAGE(Position1L));
    XmString label_string = StringCreate(MESSAGE(JobNameL));
    Widget p = XtParent(parent->BaseWidget());
-   Pixel bg;
+   XtArgVal bg;
    XtVaGetValues(p, XmNbackground, &bg, NULL);
    _w = XtVaCreateManagedWidget("DtDetailsLabel", iconWidgetClass, p,
                                GuiNsuperNode, parent->BaseWidget(),
index 4e0853bd0ec5cabdf0293fda4521a46d97b4ed96..1c0631c5f8cdc87a88d5fb80e3defa86d6bf522b 100644 (file)
@@ -191,6 +191,7 @@ void DtDND::GetDragPixmaps()
 
    Dimension height, width;
    unsigned int w, h, junk;
+   Window junkwin;
 
    XtVaGetValues(icon->BaseWidget(), XmNbackground, &bg, XmNforeground, &fg,
                  GuiNselectColor, &selectColor, XmNalignment, &alignment,
@@ -219,7 +220,7 @@ void DtDND::GetDragPixmaps()
       icon->GetPixmaps(icon->BaseWidget(), iconFile, &tmp_pixmap, &tmp_mask);
       if (tmp_pixmap && tmp_pixmap != XmUNSPECIFIED_PIXMAP)
        {
-        XGetGeometry(icon->display, tmp_pixmap, (Window *) &junk, 
+        XGetGeometry(icon->display, tmp_pixmap, &junkwin,
                      (int *) &junk, (int *) &junk, &w, &h, &junk, &junk);
         p_w = w;
         p_h = h;
index e4f9a19cf0ff049ad94443f176ac1b1d55b78dbe..123e8162b15396c523ca9a3175cbc8d1d6364601 100644 (file)
@@ -454,7 +454,7 @@ IconClassRec iconClassRec =
     },
     {
     /* icon_class record     */
-    (int) NULL,                         /* extension            */
+    0,                                  /* extension            */
     }
 };
 
@@ -1052,6 +1052,7 @@ CalculateSize(
    )
 {
     unsigned int width, height, junk;
+    Window junkwin;
     int x, y;
     Boolean show_fields = False;
     Dimension _width;
@@ -1060,7 +1061,7 @@ CalculateSize(
       {
        if (Pixmap(w) != XmUNSPECIFIED_PIXMAP)
          {
-            XGetGeometry(XtDisplay(w), Pixmap(w), (Window *) &junk,
+            XGetGeometry(XtDisplay(w), Pixmap(w), &junkwin,
                (int *) &junk, (int *) &junk, &width, &height, &junk, &junk);
             PixmapWidth(w) = (Dimension) width;
             PixmapHeight(w) = (Dimension) height;
@@ -1361,7 +1362,7 @@ CalculateSize(
       {
        Dimension x_offset, y_offset;
 
-        XGetGeometry(XtDisplay(w), StatePixmap(w), (Window *) &junk,
+        XGetGeometry(XtDisplay(w), StatePixmap(w), &junkwin,
             (int *) &junk, (int *) &junk, &width, &height, &junk, &junk);
         StatePixmapWidth(w) = (Dimension) width;
         StatePixmapHeight(w) = (Dimension) height;