[Octave-bug-tracker] [bug #53392] [GUI] Octave crashes when moving Documentation pane within GUI onto another pane

[Dan Sebald]

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

I've experimented on KDE a bit; didn't resolve much, but maybe I have a little
clearer picture.

I find that this isn't limited to Documentation widget.  It seems that any
combination of windows being tabbified will cause the blue shade and crash.

I also noticed that the Variable Editor doesn't allow the sub-panels to be
moved, no less tabbified.  But just a few moments ago, I read the Qt
documentation a little more closely and see that the following is deprecated:


QDockWidget::AllDockWidgetFeatures

DockWidgetClosable | DockWidgetMovable | DockWidgetFloatable

(Deprecated) The dock widget can be closed, moved, and floated. Since new
features might be added in future releases, the look and behavior of dock
widgets might change if you use this flag. Please specify individual flags
instead.


In the patches for Bug #53276 I've been replacing:


-    setFeatures (QDockWidget::DockWidgetMovable); // not floatable or
closeable


with


+
+    setFeatures (QDockWidget::AllDockWidgetFeatures);


I assume that on KDE, QDockWidget::AllDockWidgetFeatures does not include
DockWidgetMovable.  OK, so that I can fix easily in the Bug #53276 patch.

Out of curiosity, I changed the following from main-window.cc:


        setDockOptions (QMainWindow::AnimatedDocks
                        | QMainWindow::AllowNestedDocks
                        | QMainWindow::AllowTabbedDocks);


to


        setDockOptions (QMainWindow::AllowNestedDocks
                        | QMainWindow::AllowTabbedDocks);


and get no crash in KDE.

I did put in a bunch of debugging comments to see where things might be
crashing.  However, it seems right away, within Qt and not any of the Octave
code, the program hangs.  The patch from Bug #53276 does not crash, even with
the animated docks.  I suspect it is because of some settings change tied into
the above options, or perhaps not restricting to just
QDockWidget::DockWidgetMovable but the patch now using the default float and
close buttons.

I will investigate more when I have time.

    _______________________________________________________

Reply to this item at:

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

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