Review Request: Save/Restore the ProjectTreeView. This enables a more seamless transition when returning to a project that was closed.

[Andrew Fuller]

> On Jan. 8, 2012, 3:39 p.m., Milian Wolff wrote:
> > ah well, just the default dtor won't make it abi-stable as we'd have to change the inheritance probably, or?
> > 
> > anyhow, I don't see a reason to make the saver public api to begin with. Could you move it to plugins/projectmanagerview ? or why should it be public in your opinion?
> > 
> > thanks

The change to a default ctor was alluding to the fact that in order to use a class as template argument for KViewStateMaintainer it must have a default constructor.  I'm not sure why the inheritance would ever possibly change...?

I agree with you that it is unlikely anyone would have use for ProjectModelSaver exported.  I've moved it to plugins/projectmanagerview.


- Andrew


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


On Jan. 8, 2012, 6:59 p.m., Andrew Fuller wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103619/
> -----------------------------------------------------------
> 
> (Updated Jan. 8, 2012, 6:59 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> Save/Restore the ProjectTreeView.  This enables a more seamless transition when returning to a project that was closed.
> 
> 
> Diffs
> -----
> 
>   plugins/projectmanagerview/CMakeLists.txt 7b58e8e5489819b0370f5d55e0218f063da90af2 
>   plugins/projectmanagerview/projectmodelsaver.h PRE-CREATION 
>   plugins/projectmanagerview/projectmodelsaver.cpp PRE-CREATION 
>   plugins/projectmanagerview/projecttreeview.h 16893f0713b83a78a5b025a55fcfa082265d4419 
>   plugins/projectmanagerview/projecttreeview.cpp 8237af513627c0459d571de2bccc275e8a4b0ae9 
>   shell/sessioncontroller.h 551894d4131218b9ab2640e28f2c354ce319a105 
>   shell/sessioncontroller.cpp d2ee8732710e0e4bded56e547d83a3906b20b018 
> 
> Diff: http://git.reviewboard.kde.org/r/103619/diff/diff
> 
> 
> Testing
> -------
> 
> Load a session, Exit, and return.  Voilà!  The tree is right where you left it.
> 
> 
> Thanks,
> 
> Andrew Fuller
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120108/ac0d2fae/attachment.html>