home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1993 #3 / NN_1993_3.iso / spool / comp / windows / x / 21505 < prev    next >
Encoding:
Text File  |  1993-01-23  |  4.4 KB  |  176 lines
  1. Newsgroups: comp.windows.x
  2. Path: sparky!uunet!cs.utexas.edu!hermes.chpc.utexas.edu!news.utdallas.edu!corpgate!bnrgate!bnr.co.uk!bmdhh243!bcars267!spock
  3. From: spock@bnr.ca (Spock)
  4. Subject: XRN with OSF/MOTIF 1.2.1
  5. Message-ID: <1993Jan22.223528.12980@bnr.ca>
  6. Sender: news@bnr.ca (usenet)
  7. Nntp-Posting-Host: wipeout
  8. Organization: Bell-Northern Research, Ontario, Canada
  9. Date: Fri, 22 Jan 1993 22:35:28 GMT
  10. Lines: 164
  11.  
  12. Hi,
  13.  
  14.    If you don't use the motif version of xrn its time to skip to the
  15.    next article.
  16.  
  17.    It seems that a minor change between Motif 1.2.0 and 1.2.1 causes
  18.    a segv when scrolling an article.  The new version of Motif tries
  19.    to dereference through the "event" passed to XtCallActionProc().
  20.  
  21.    To cure this, I just passed a valid event pointer instead of "0".
  22.  
  23.    The following context diff should be applied on top of xrn6-17 at
  24.    patch level 2.  It is backward compatible to any release of Motif.
  25.  
  26.    I have sent this patch to 'xrn@ic.berkeley.edu', and with any luck
  27.    it will appear soon as official patch #3.
  28.  
  29. Enjoy,
  30. R
  31.  
  32. ---------------------------------------------------------------------
  33.  
  34. *** xrn6-17/buttons.c    Fri Jan 22 17:06:26 1993
  35. --- xrn/buttons.c    Fri Jan 22 16:33:12 1993
  36. ***************
  37. *** 4197,4206 ****
  38.    * called when the user wants to scroll the current article
  39.    */
  40.   {
  41.       if (Mode != ARTICLE_MODE) {
  42.       return;
  43.       }
  44. !     XtCallActionProc(ArticleText, "next-page", 0, 0, 0);
  45.       return;
  46.   }
  47.   
  48. --- 4197,4209 ----
  49.    * called when the user wants to scroll the current article
  50.    */
  51.   {
  52. +     XEvent    event;
  53. + 
  54.       if (Mode != ARTICLE_MODE) {
  55.       return;
  56.       }
  57. !     bzero( (char *) &event, sizeof( XEvent ) );
  58. !     XtCallActionProc(ArticleText, "next-page", &event, 0, 0);
  59.       return;
  60.   }
  61.   
  62. ***************
  63. *** 4213,4222 ****
  64.    * called when the user wants to scroll the current article
  65.    */
  66.   {
  67.       if (Mode != ARTICLE_MODE) {
  68.       return;
  69.       }
  70. !     XtCallActionProc(ArticleText, "previous-page", 0, 0, 0);
  71.       return;
  72.   }
  73.   
  74. --- 4216,4228 ----
  75.    * called when the user wants to scroll the current article
  76.    */
  77.   {
  78. +     XEvent    event;
  79. + 
  80.       if (Mode != ARTICLE_MODE) {
  81.       return;
  82.       }
  83. !     bzero( (char *) &event, sizeof( XEvent ) );
  84. !     XtCallActionProc(ArticleText, "previous-page", &event, 0, 0);
  85.       return;
  86.   }
  87.   
  88. ***************
  89. *** 4229,4238 ****
  90.    * called when the user wants to scroll the current article
  91.    */
  92.   {
  93.       if (Mode != ARTICLE_MODE) {
  94.       return;
  95.       }
  96. !     XtCallActionProc(ArticleText, "scroll-one-line-up", 0, 0, 0);
  97.       return;
  98.   }
  99.   
  100. --- 4235,4247 ----
  101.    * called when the user wants to scroll the current article
  102.    */
  103.   {
  104. +     XEvent    event;
  105. + 
  106.       if (Mode != ARTICLE_MODE) {
  107.       return;
  108.       }
  109. !     bzero( (char *) &event, sizeof( XEvent ) );
  110. !     XtCallActionProc(ArticleText, "scroll-one-line-up", &event, 0, 0);
  111.       return;
  112.   }
  113.   
  114. ***************
  115. *** 4245,4254 ****
  116.    * called when the user wants to scroll the current article
  117.    */
  118.   {
  119.       if (Mode != ARTICLE_MODE) {
  120.       return;
  121.       }
  122. !     XtCallActionProc(ArticleText, "scroll-one-line-down", 0, 0, 0);
  123.       return;
  124.   }
  125.   
  126. --- 4254,4266 ----
  127.    * called when the user wants to scroll the current article
  128.    */
  129.   {
  130. +     XEvent    event;
  131. + 
  132.       if (Mode != ARTICLE_MODE) {
  133.       return;
  134.       }
  135. !     bzero( (char *) &event, sizeof( XEvent ) );
  136. !     XtCallActionProc(ArticleText, "scroll-one-line-down", &event, 0, 0);
  137.       return;
  138.   }
  139.   
  140. ***************
  141. *** 4261,4266 ****
  142. --- 4273,4280 ----
  143.    * called when the user wants to scroll the current article
  144.    */
  145.   {
  146. +     XEvent    event;
  147. + 
  148.       if (Mode != ARTICLE_MODE) {
  149.       return;
  150.       }
  151. ***************
  152. *** 4274,4280 ****
  153.   
  154.         data = XmTextGetString(ArticleText);
  155.         XmTextShowPosition(ArticleText, strlen(data)-1);
  156. !       XtCallActionProc(ArticleText, "scroll-one-line-up", 0, 0, 0);
  157.         XtFree(data);
  158.       }
  159.   #endif
  160. --- 4288,4295 ----
  161.   
  162.         data = XmTextGetString(ArticleText);
  163.         XmTextShowPosition(ArticleText, strlen(data)-1);
  164. !       bzero( (char *) &event, sizeof( XEvent ) );
  165. !       XtCallActionProc(ArticleText, "scroll-one-line-up", &event, 0, 0);
  166.         XtFree(data);
  167.       }
  168.   #endif
  169.  
  170. ---------------------------------------------------------------------
  171.  
  172. -- 
  173.                         "At least I'm not wasting my life as a nameless,
  174.                          faceless droid in some software megafactory..."
  175.                                                            - anonymous
  176.