home *** CD-ROM | disk | FTP | other *** search
/ PC World Komputer 1999 March / PCWK3A99.iso / Linux / DDD331 / DDD-3_1_.000 / DDD-3_1_ / ddd-3.1.1 / patches / lesstif-0.83.patch < prev    next >
Encoding:
Text File  |  1998-03-20  |  15.1 KB  |  499 lines
  1. # $Id: lesstif-0.83.patch,v 1.1 1998/03/20 12:08:33 zeller Exp $
  2.  
  3. This file contains a variety of patches for lesstif-0.83.  Although
  4. none of them is really serious, we suggest that these patches be
  5. applied (using `patch -l < lesstif-0.83.patch') before building
  6. LessTif and running DDD with LessTif.
  7.  
  8.  
  9. Subject: LessTif 0.83: Modifiers cause XBell on read-only Text, and a patch
  10. To: lesstif@hungry.com (LessTif Mailing List)
  11. Date: Sun, 15 Mar 1998 17:43:52 +0100 (MET)
  12.  
  13. Hi!
  14.  
  15. The following problem occurs in LessTif 0.83 and earlier:
  16.  
  17. * Pressing a modifier while the pointer is over a read-only Text
  18.   widget causes the X bell to be rung.
  19.  
  20. The enclosed patch fixes this.
  21.  
  22. Thanks for your work,
  23.  
  24. -- 
  25. Andreas Zeller     Technische Universitaet Braunschweig, Germany
  26.                    mailto:zeller@acm.org http://www.cs.tu-bs.de/~zeller/
  27. *** lesstif-0.83/lib/Xm/TextIn.c.orig       Sun Mar 15 17:33:39 1998
  28. --- lesstif-0.83/lib/Xm/TextIn.c    Sun Mar 15 17:35:20 1998
  29. ***************
  30. *** 543,552 ****
  31. --- 543,558 ----
  32.       if (len <= 0)
  33.       {
  34.         return;
  35.       }
  36.   
  37. +     if (!Text_Editable(w))
  38. +     {
  39. +       VerifyBell(w);
  40. +       return;
  41. +     }
  42. + 
  43.       (*Text_Output(w)->DrawInsertionPoint) (w, Text_CursorPos(w), off);
  44.   
  45.       blockrec.ptr = XtMalloc(len + 1);
  46.       blockrec.length = len;
  47.       blockrec.format = XmFMT_8_BIT;
  48. ***************
  49. *** 1889,1904 ****
  50.       char buf[INSERTCHARBUFSIZ];
  51.       KeySym keysym;
  52.       int len, status;
  53.   
  54.       DEBUGOUT(XdbDebug(__FILE__, w, "SelfInsert"));
  55. - 
  56. -     if (!Text_Editable(w))
  57. -     {
  58. -       VerifyBell((XmTextWidget)w);
  59. -       return;
  60. -     }
  61.   
  62.       len = XmImMbLookupString((Widget)w, (XKeyPressedEvent *)ev, buf, INSERTCHARBUFSIZ, &keysym, &status);
  63.   
  64.       if (XdbInDebug(__FILE__, w))
  65.       {
  66. --- 1895,1904 ----
  67. *** lesstif-0.83/lib/Xm/TextF.c.orig        Fri Mar 13 18:43:10 1998
  68. --- lesstif-0.83/lib/Xm/TextF.c     Sun Mar 15 17:39:23 1998
  69. ***************
  70. *** 2714,2723 ****
  71. --- 2714,2734 ----
  72.   }
  73.   
  74.   static void
  75.   DoInsert(XmTextFieldWidget w, XEvent *ev, char *buf, int len)
  76.   {
  77. +     if (len <= 0)
  78. +     {
  79. +       return;
  80. +     }
  81. + 
  82. +     if (!TextF_Editable(w))
  83. +     {
  84. +       VerifyBell(w);
  85. +       return;
  86. +     }
  87. + 
  88.       if (XdbInDebug(__FILE__, (Widget)w))
  89.       {
  90.       char *tmp;
  91.   
  92.         /* buf is not necessarily NULL terminated!!!! */
  93. ***************
  94. *** 4029,4047 ****
  95.       char buf[INSERTCHARBUFSIZ];
  96.       KeySym keysym;
  97.       int len, status;
  98.   
  99.       DBGW("self_insert");
  100. - 
  101. -     if (!TextF_Editable(tfw))
  102. -     {
  103. -       if (TextF_VerifyBell(tfw))
  104. -       {
  105. -           beep((Widget)tfw, ev, params, num_params);
  106. -       }
  107. -       return;
  108. -     }
  109.   
  110.       len = XmImMbLookupString((Widget)tfw,
  111.                              (XKeyPressedEvent *)ev, buf,
  112.                              INSERTCHARBUFSIZ,
  113.                              &keysym,
  114. --- 4040,4049 ----
  115.  
  116. Subject: XmTrackingEvent() bugs in LessTif 0.83, and a patch
  117. To: lesstif@hungry.com (LessTif Mailing List)
  118. Date: Tue, 17 Mar 1998 12:39:23 +0100 (MET)
  119.  
  120. Hi!
  121.  
  122. I reported two bugs in LessTif 0.83 XmTrackingEvent(); the enclosed
  123. patch fixes them.
  124.  
  125. >  In XmTrackingEvent(), a KeyRelease event causes the tracking to be
  126. >  finished right away.  This has a disadvantage: if I invoke
  127. >  XmTrackingEvent using some key (say, F1), the tracking is finished
  128. >  as soon as the key is released.  In M*tif, AFAIK, tracking is not
  129. >  influenced by key events.  Suggestion: check for ButtonRelease
  130. >  events only.
  131.  
  132. The patch modifies XmTrackingEvent() such that the tracking is
  133. finished only if a key is first pressed, then released, in accordance
  134. with the Motif 1.2 spec.
  135.  
  136. > In LessTif 0.83 (TrackLoc.c), XmTrackingEvent() does not return
  137. > anything in event_return.
  138.  
  139. The patch fixes this problem, too.
  140.  
  141. The patch also fixes the following improvements:
  142.  
  143. * If the pointer grab fails, issues an error message
  144. * Use XtLastTimestampProcessed() instead of CurrentTime for grabs
  145.  
  146. Happy hacking,
  147. -- 
  148. Andreas Zeller     Technische Universitaet Braunschweig, Germany
  149.                    mailto:zeller@acm.org http://www.cs.tu-bs.de/~zeller/
  150. *** lesstif-0.83/lib/Xm/TrackLoc.c.orig     Tue Mar 17 12:09:15 1998
  151. --- lesstif-0.83/lib/Xm/TrackLoc.c  Tue Mar 17 12:25:21 1998
  152. ***************
  153. *** 34,43 ****
  154. --- 34,45 ----
  155.   XmTrackingEvent(Widget widget, Cursor cursor,
  156.                 Boolean confine_to, XEvent *event_return)
  157.   {
  158.       Window confine_to_this;
  159.       XEvent ev;
  160. +     Boolean key_pressed = False;
  161. +     Time time;
  162.   
  163.       if (confine_to)
  164.       {
  165.         confine_to_this = XtWindow(widget);
  166.       }
  167. ***************
  168. *** 46,75 ****
  169.         confine_to_this = None;
  170.       }
  171.   
  172.       DEBUGOUT(XdbDebug(__FILE__, widget, "XmTrackingEvent: XtGrabPointer()\n"));
  173.   
  174. !     XtGrabPointer(widget,
  175. !                 True,
  176. !                 ButtonReleaseMask | ButtonPressMask,
  177. !                 GrabModeAsync,
  178. !                 GrabModeAsync,
  179. !                 confine_to_this,
  180. !                 cursor,
  181. !                 CurrentTime);
  182.   
  183.       while (True)
  184.       {
  185.         XtAppNextEvent(XtWidgetToApplicationContext(widget), &ev);
  186.   
  187. !       if (ev.xbutton.type == KeyRelease ||
  188. !           (ev.xbutton.type == ButtonRelease && ev.xbutton.button == 1))
  189.         {
  190.             DEBUGOUT(XdbDebug(__FILE__, widget,
  191.                               "XmTrackingEvent: XtUngrabPointer()\n"));
  192.   
  193. !           XtUngrabPointer(widget, CurrentTime);
  194.   
  195.             /* If the button was clicked outside of this programs windows,
  196.              * the widget that grabbed the pointer will get the event.  So,
  197.              * we check the bounds of the widget against the coordinates of
  198.              * the event.  If they're outside, we return NULL.  Otherwise we
  199. --- 48,88 ----
  200.         confine_to_this = None;
  201.       }
  202.   
  203.       DEBUGOUT(XdbDebug(__FILE__, widget, "XmTrackingEvent: XtGrabPointer()\n"));
  204.   
  205. !     time = XtLastTimestampProcessed(XtDisplay(widget));
  206. !     if (XtGrabPointer(widget, True,
  207. !                     ButtonReleaseMask | ButtonPressMask,
  208. !                     GrabModeAsync, GrabModeAsync,
  209. !                     confine_to_this, cursor, time) != GrabSuccess)
  210. !     {
  211. !       _XmWarning(widget, "XmTrackingEvent: Could not grab pointer");
  212. !       return NULL;
  213. !     }
  214.   
  215.       while (True)
  216.       {
  217.         XtAppNextEvent(XtWidgetToApplicationContext(widget), &ev);
  218. +       time = XtLastTimestampProcessed(XtDisplay(widget));
  219.   
  220. !       if (ev.xbutton.type == KeyPress)
  221.         {
  222. +           /* Avoid exiting upon releasing the key that caused
  223. +                XmTrackingEvent() to be invoked */
  224. +           key_pressed = True;
  225. +       }
  226. +       else if ((ev.xbutton.type == KeyRelease && key_pressed) ||
  227. +                (ev.xbutton.type == ButtonRelease && ev.xbutton.button == 1))
  228. +       {
  229. +           if (event_return != NULL)
  230. +               *event_return = ev;
  231. + 
  232.             DEBUGOUT(XdbDebug(__FILE__, widget,
  233.                               "XmTrackingEvent: XtUngrabPointer()\n"));
  234.   
  235. !           XtUngrabPointer(widget, time);
  236.   
  237.             /* If the button was clicked outside of this programs windows,
  238.              * the widget that grabbed the pointer will get the event.  So,
  239.              * we check the bounds of the widget against the coordinates of
  240.              * the event.  If they're outside, we return NULL.  Otherwise we
  241.  
  242. Subject: An XmCvtXmStringToText() implementation
  243. To: lesstif@hungry.com (LessTif Mailing List)
  244. Date: Tue, 17 Mar 1998 15:23:39 +0100 (MET)
  245.  
  246. Hi!
  247.  
  248. I want to contribute an implementation of the XmCvtXmStringToText()
  249. function - a XmString -> String converter.  Among others, this
  250. converter is useful in creating the titles of tear-off menus (see
  251. second mail).
  252.  
  253. Greetings,
  254.  
  255. -- 
  256. Andreas Zeller     Technische Universitaet Braunschweig, Germany
  257.                    mailto:zeller@acm.org http://www.cs.tu-bs.de/~zeller/
  258.  
  259. *** lesstif-0.83/lib/Xm/ResConvert.c.orig   Tue Mar 17 15:18:35 1998
  260. --- lesstif-0.83/lib/Xm/ResConvert.c        Tue Mar 17 15:16:42 1998
  261. ***************
  262. *** 908,919 ****
  263.   Boolean
  264.   XmCvtXmStringToText(Display *display, XrmValuePtr args,
  265.                     Cardinal *num_args, XrmValue *from_val, XrmValue *to_val,
  266.                     XtPointer *converter_data)
  267.   {
  268. !     DEBUGOUT(XdbDebug(__FILE__, NULL, "XmCvtXmStringToText\n"));
  269. !     return False;
  270.   }
  271.   
  272.   /*
  273.    * Compare two strings to see if they're equivalent. This one is used
  274.    * whenever comparing motif constants with resource string values. The
  275. --- 908,980 ----
  276.   Boolean
  277.   XmCvtXmStringToText(Display *display, XrmValuePtr args,
  278.                     Cardinal *num_args, XrmValue *from_val, XrmValue *to_val,
  279.                     XtPointer *converter_data)
  280.   {
  281. !     char *value = strdup("");
  282. !     XmString m = (XmString)from_val->addr;
  283. !     XmStringContext c;
  284. !     XmStringComponentType t = XmSTRING_COMPONENT_UNKNOWN;
  285. ! 
  286. !     if (*num_args != 0)
  287. !     {
  288. !       XtWarningMsg("wrongParameters", "XmCvtXmStringToText",
  289. !                    "XtToolkitError",
  290. !                    "XmString to String conversion needs no extra arguments",
  291. !                    (String *)NULL, (Cardinal *)NULL);
  292. !     }
  293. ! 
  294. !     XmStringInitContext(&c, m);
  295. ! 
  296. !     while (t != XmSTRING_COMPONENT_END)
  297. !     {
  298. !       char *s_text            = NULL;
  299. !       XmStringCharSet s_cs    = NULL;
  300. !       XmStringDirection d     = XmSTRING_DIRECTION_DEFAULT;
  301. !       XmStringComponentType u = XmSTRING_COMPONENT_UNKNOWN;
  302. !       unsigned short ul       = 0;
  303. !       unsigned char *s_uv     = NULL;
  304. !       char *text              = NULL;
  305. !       
  306. !       t = XmStringGetNextComponent(c, &s_text, &s_cs, &d, &u, &ul, &s_uv);
  307. ! 
  308. !       switch (t)
  309. !       {
  310. !       case XmSTRING_COMPONENT_TEXT:
  311. !       case XmSTRING_COMPONENT_LOCALE_TEXT:
  312. !           text = s_text;
  313. !           break;
  314. ! 
  315. !       case XmSTRING_COMPONENT_SEPARATOR:
  316. !           text = "\n";
  317. !           break;
  318. ! 
  319. !       default:
  320. !           /* Unknown component */
  321. !           text = NULL;
  322. !           break;
  323. !       }
  324. ! 
  325. !       if (text != NULL)
  326. !       {
  327. !           value = XtRealloc(value, strlen(value) + strlen(text) + 1);
  328. !           strcat(value, text);
  329. !       }
  330. ! 
  331. !       XtFree(s_text);
  332. !       XtFree(s_cs);
  333. !       XtFree((char *)s_uv);
  334. !     }
  335. ! 
  336. !     to_val->addr = value;
  337. !     to_val->size = strlen(value);  /* FIXME: should this be sizeof(value)? */
  338. ! 
  339. !     DEBUGOUT(XdbDebug(__FILE__, NULL,
  340. !                     "XmCvtXmStringToText(%p) => %s\n",
  341. !                     from_val->addr, to_val->addr));
  342. ! 
  343. !     return True;
  344.   }
  345.   
  346.   /*
  347.    * Compare two strings to see if they're equivalent. This one is used
  348.    * whenever comparing motif constants with resource string values. The
  349.  
  350. Subject: An improved method for determining tear-off titles
  351. To: lesstif@hungry.com (LessTif Mailing List)
  352. Date: Tue, 17 Mar 1998 15:26:41 +0100 (MET)
  353.  
  354. Hi!
  355.  
  356. The default implementation of tear-off menus uses XmStringGetLtoR() to
  357. copy the label string to the tear-off menu title.  This fails on a
  358. number of occasions - if the label does not use a standard font, or
  359. uses accelerators, for example.  The included patch makes tear-off
  360. menus use the XmCvtXmStringToText() converter instead; an
  361. implementation is given in a second mail.
  362.  
  363. Greetings,
  364.  
  365. -- 
  366. Andreas Zeller     Technische Universitaet Braunschweig, Germany
  367.                    mailto:zeller@acm.org http://www.cs.tu-bs.de/~zeller/
  368.  
  369. *** lesstif-0.83/lib/Xm/TearOff.c.orig      Tue Mar 17 15:18:21 1998
  370. --- lesstif-0.83/lib/Xm/TearOff.c   Tue Mar 17 15:16:34 1998
  371. ***************
  372. *** 352,378 ****
  373.   
  374.       sprintf(to_shell_name, "%s%s",
  375.                 RC_CascadeBtn(w) ? XtName(RC_CascadeBtn(w)) : "",
  376.                 TEAR_OFF_SHELL_NAME);
  377.       {
  378. !     XmString string;
  379. !     unsigned char type;
  380.   
  381.         if ( RC_Type(w) != XmMENU_POPUP)
  382.         {
  383.             XtVaGetValues(RC_CascadeBtn(w),
  384.                     XmNlabelString, &string,
  385.                     XmNlabelType, &type,
  386.                     NULL);
  387.             if (type == XmSTRING)
  388.             {
  389. !               XmStringGetLtoR(string, XmFONTLIST_DEFAULT_TAG, &to_shell_title);
  390.                 XmStringFree(string);
  391.             }
  392.             else
  393.             {
  394. !                   to_shell_title = strdup("");
  395.             }
  396.         }
  397.         else
  398.         {
  399.                 to_shell_title = strdup("");
  400. --- 352,394 ----
  401.   
  402.       sprintf(to_shell_name, "%s%s",
  403.                 RC_CascadeBtn(w) ? XtName(RC_CascadeBtn(w)) : "",
  404.                 TEAR_OFF_SHELL_NAME);
  405.       {
  406. !       XmString string;
  407. !       unsigned char type;
  408.   
  409.         if ( RC_Type(w) != XmMENU_POPUP)
  410.         {
  411.             XtVaGetValues(RC_CascadeBtn(w),
  412.                     XmNlabelString, &string,
  413.                     XmNlabelType, &type,
  414.                     NULL);
  415.             if (type == XmSTRING)
  416.             {
  417. !               /* Invoke XmString -> String converter */
  418. !               Cardinal zero = 0;
  419. !               XrmValue from;
  420. !               XrmValue to;
  421. !               from.addr = (char *)string;
  422. ! 
  423. !               if (XmCvtXmStringToText(XtDisplay(w), NULL, &zero, 
  424. !                                       &from, &to, NULL))
  425. !               {
  426. !                   to_shell_title = to.addr;
  427. !               }
  428. !               else
  429. !               {
  430. !                   /* converter failed - use simple fallback */
  431. !                   XmStringGetLtoR(string, XmFONTLIST_DEFAULT_TAG, 
  432. !                                   &to_shell_title);
  433. !               }
  434.                 XmStringFree(string);
  435.             }
  436.             else
  437.             {
  438. !               to_shell_title = strdup("");
  439.             }
  440.         }
  441.         else
  442.         {
  443.                 to_shell_title = strdup("");
  444.  
  445. Subject: Button3 popup menus are insensitive the first time they pop up, 
  446.      and a patch
  447. To: lesstif@hungry.com (LessTif Mailing List)
  448. Date: Fri, 20 Mar 1998 12:49:58 +0100 (MET)
  449.  
  450. Hi!
  451.  
  452. In LessTif 0.83, a popup menu bound to Button3 is insensitive the
  453. first time they pop up - the menu is not placed in drag mode, as it
  454. should be.  The enclosed patch to RowColumn.c:_XmPostPopupMenu() fixes
  455. this problem; in the long run, one should care to pass a correct event
  456. structure to _XmPostPopupMenu().
  457.  
  458. This was the last outstanding problem with DDD and LessTif 0.83.  You
  459. guys surely have done a great work.
  460.  
  461. Happy hacking,
  462.  
  463. -- 
  464. Andreas Zeller     Technische Universitaet Braunschweig, Germany
  465.                    mailto:zeller@acm.org http://www.cs.tu-bs.de/~zeller/
  466. *** lesstif-0.83/lib/Xm/RowColumn.c.orig    Fri Mar 13 23:25:33 1998
  467. --- lesstif-0.83/lib/Xm/RowColumn.c    Fri Mar 20 13:06:51 1998
  468. ***************
  469. *** 3050,3060 ****
  470.       DEBUGOUT(XdbDebug(__FILE__, wid, "PostPopupMenu()\n"));
  471.   
  472.       time = XtLastTimestampProcessed(XtDisplay(wid));
  473.       if (type == 0)
  474.       {
  475. !     type = KeyPress;
  476.       }
  477.   
  478.       state->RC_ButtonEventStatus.waiting_to_be_managed = False;
  479.       state->RC_ButtonEventStatus.time = time;
  480.   
  481. --- 3050,3066 ----
  482.       DEBUGOUT(XdbDebug(__FILE__, wid, "PostPopupMenu()\n"));
  483.   
  484.       time = XtLastTimestampProcessed(XtDisplay(wid));
  485.       if (type == 0)
  486.       {
  487. !     /* Was KeyPress in LessTif 0.83, but this makes button3 popup
  488. !            menus insensitive the first time they pop up - they are not
  489. !            placed in drag mode.
  490. !        - Andreas Zeller <zeller@ips.cs.tu-bs.de>
  491. !        FIXME: Don't know whether this breaks keypress popup menus. */
  492. !        
  493. !     type = ButtonPress;
  494.       }
  495.   
  496.       state->RC_ButtonEventStatus.waiting_to_be_managed = False;
  497.       state->RC_ButtonEventStatus.time = time;
  498.   
  499.