[Octave-bug-tracker] [bug #37234] GUI fails to properly call all destructors and free all memory

[Torsten]

Follow-up Comment #10, bug #37234 (project octave):

I agree, ownership should be taken via addItem that in turn is called by
addWidget. Strangely enough, _current_directory_combo_box can be deleted
without error (only a second delete leads to a segmentation fault). 

Two other things:
Is the test for 0 before deleting necessary? The c++ standard says that
nothing happens if delete gets a null pointer.
Should we set the pointers to 0 afterwards in order to be sure a second delete
would be harmless? This would allow to delete _current_directory_combo_box
(just to be sure) 


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?37234>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/