[MassifVisualizer] Review Request 113523: Implementation of multiple document interface

[Arnold Dumas]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113523/
-----------------------------------------------------------

(Updated Nov. 3, 2013, 8:16 p.m.)


Review request for Massif Visualizer.


Changes
-------

There is still one minor issue with this new patch but I first would like to find an agreement on the design first. Thus It will then be possible to fix the remainings bugs.
Instead of using ThreadWeaver, wich doesn't work, at least for me, I've used "raw" ParseWorker and QThreads. All the signals & slots mapping is done manually but it works fine so far. As far as I know, the design if I had use ThreadWeaver would have been the same. Hence It should no be too hard to move to ThreadWeaver later on.
Plus I've moved allmost all the code related to kgraphviewer in the DocumentWidget class. It should work but I didn't test is so far as I'm unable to compile kgraphviewer:master.
I hope this patch will fullfil your ideas about how multiple document interface should be designed and implemented. ;)


Repository: massif-visualizer


Description
-------

All the former logic of creating the charts, the legend ... etc is now contained in the MassifDocumentWidget.
This widget is instanciated by the top-most tab widget. When the active tab changes, the massifDocumentChanged slot is called.
Some actions are enabled/disabled regarding of the situation. I noticed no particular bug or misbehavior when testing.
One more thing needs to be done: parallel file loading using ThreadWeaver.


Diffs (updated)
-----

  app/CMakeLists.txt 31290f6 
  app/documentwidget.h PRE-CREATION 
  app/documentwidget.cpp PRE-CREATION 
  app/main.cpp be938b4 
  app/mainwindow.h 2acc6a8 
  app/mainwindow.cpp b33ff60 
  app/mainwindow.ui 8ac4119 

Diff: http://git.reviewboard.kde.org/r/113523/diff/


Testing
-------


Thanks,

Arnold Dumas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/massif-visualizer/attachments/20131103/9f0b8d1d/attachment.html>