Review Request: Save workingset state per area

Nicolas Alvarez nicolas.alvarez at gmail.com
Fri Jun 17 23:30:10 UTC 2011


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



shell/workingsets/workingset.cpp
<http://git.reviewboard.kde.org/r/101589/#comment3263>

    Any reason why the KConfigGroups are passed by value and not by ref?



sublime/areaindex.h
<http://git.reviewboard.kde.org/r/101589/#comment3264>

    Doxygen and function prototype don't agree in parameter name.


- Nicolas


On June 12, 2011, 3:01 a.m., Olivier Jean de Gaalon wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101589/
> -----------------------------------------------------------
> 
> (Updated June 12, 2011, 3:01 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> This patch saves and loads certain area-specific states for each working set, in particular, it saves
> *The document state (cursor position, or selection)
> *The currently active document
> *The order of the documents
> For each area, so that when you reopen KDevelop, all is as you left it, in each area.
> 
> Also, this fixes a bug where when you split a view and then unsplit it (close the split off view), when you restarted KDevelop it would come back (unless you closed the entire working set or closed the split view in *all* the areas).
> 
> If this fixes any reported bugs, let me know what they are.
> 
> It's possible this isn't the best way to do this as I'm not familiar with this area at all, but it's unlikely to break horribly, and is even backwards compatible (ie, you won't lose your current workingsets).
> 
> 
> Diffs
> -----
> 
>   shell/workingsets/workingset.h 512c5cb 
>   shell/workingsets/workingset.cpp c494e51 
>   sublime/areaindex.h 7dfc13f 
> 
> Diff: http://git.reviewboard.kde.org/r/101589/diff
> 
> 
> Testing
> -------
> 
> Starting and stopping kdevelop, splitting and unsplitting, restarting kdevelop, reordering documents, changing cursor positions, restarting kdevelop...
> 
> 
> Thanks,
> 
> Olivier Jean de
> 
>

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


More information about the KDevelop-devel mailing list