home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1993 #3 / NN_1993_3.iso / spool / comp / windows / x / motif / 8810 < prev    next >
Encoding:
Text File  |  1993-01-25  |  3.3 KB  |  102 lines
  1. Path: sparky!uunet!mcsun!uknet!lsl!snail
  2. From: snail@lsl.co.uk
  3. Newsgroups: comp.windows.x.motif
  4. Subject: Re: XmSelectionBoxGetChild crashes application
  5. Message-ID: <1993Jan25.173251.2979@lsl.co.uk>
  6. Date: 25 Jan 93 17:32:51 GMT
  7. References: <5935@esf.esf.de> <1993Jan22.164824@mccall.com>
  8. Organization: Laser-Scan Ltd., Cambridge
  9. Lines: 91
  10.  
  11. In article <1993Jan22.164824@mccall.com>, tp@mccall.com (Terry Poot) writes:
  12. > 
  13. > In article <5935@esf.esf.de>, klaus@tat2.esf.de (Klaus Wicovsky) writes:
  14. >>----------------------------------------------------------------------
  15. >>void fs_okCB(w, client_data, call_data)
  16. >>...
  17. >>...
  18. >>         text=XmSelectionBoxGetChild (w, XmDIALOG_TEXT);
  19. >>    filename=XmTextGetSelection(text);
  20. >>...
  21. >>    list = XmSelectionBoxGetChild (selected, XmDIALOG_LIST_LABEL);
  22. >>----------------------------------------------------------------------
  23. >>
  24. >>Whenever this callback procedure is carried out, my application crashes at
  25. >>the last line. "w" would be my original widget for which I define this
  26. >>callback
  27. >>and "selected" would be my new widget, which would like to modify.
  28. >>
  29. >>Although "selected" is globally defined, the last statement crashes my
  30. >>application. Any  idea why this could happen ???
  31.  
  32. You don't need global variables. Globals are poor engineering. See below.
  33.  
  34. > Happened to me last week. If you check, I think you'll find that w is not the
  35. > widget id of the selection box, but of the OK button. XmSelectionBoxGetChild
  36. > appears not to check for the widget id actually being a selection box, so it
  37. > crashes with a memory access error. I don't know if XtParent would reliably get
  38. > you the widget id of the selection box, it depends on how Motif bundles all that
  39. > stuff together. You need to remember the widget id of the selection box by some
  40. > other method so it will be accessible to your callback.
  41.  
  42. The last sentence....
  43.  
  44. client_data should be used for this.
  45. For example:-
  46.  
  47.     a) Data structures
  48.  
  49. struct my_data {
  50.     MY_DATA_PTR    self;
  51.  
  52.     Widget        selected_w;
  53.  
  54.     other data....
  55. };
  56.  
  57. typedef struct my_data *MY_DATA_PTR;
  58.  
  59.     b) Allocate some memory the size of a structure you have defined.
  60.  
  61. MY_DATA_PTR    mdp;
  62.  
  63. mdp = (MY_DATA_PTR)malloc(sizeof(struct my_data));
  64. if (mdp == NULL)
  65.             /* oh shit */
  66.  
  67. mdp->self = mdp;    /* self consistency, one simple way... */
  68.  
  69.     We don't do self consistency here like this, our system is more
  70.     complex and allows us to type the objects etc.
  71.  
  72. mdp->selected_w = my_select_widget;
  73.  
  74.     /* tell your selection box callbacks to use mdp as their client data*/
  75.  
  76.     c) Whenever referencing the memory cast it to a pointer to a structure
  77.     of the type you have defined. For example in your callback routine.
  78.  
  79. static XtCallbackProc my_callback(Widget w,
  80.                                   XtPointer client_data,
  81.                                   XmAnyCallbackStruct *str)
  82. {
  83.     MY_DATA_PTR    mdp = (MY_DATA_PTR)client_data;
  84.     Widget        list_w;
  85.  
  86.     list_w = XmSelectionBoxGetChild (mdp->selected_w, XmDIALOG_LIST_LABEL);
  87.  
  88.     /* etc... */
  89.  
  90.     return NULL;    /* gcc is so fussy :-) */
  91. }
  92.     
  93. End of lesson. No C++ needed, but you've got objects now. It ain't hard, and
  94. no globals! Anyone hiring?
  95. -- 
  96. snail@lsl.co.uk      
  97.  
  98. "Washing one's hands of the conflict between the powerful and the powerless
  99.  means to side with the powerful, not to be Neutral."
  100.                                                      Quote by Freire.
  101.                                                      Poster by OXFAM.
  102.