Review Request: change default config of taskmanager

[Christoph Feck]

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


Was this change intended for 4.9 or just master as indicated in this request? Branch is actually in freeze, the bug fix says "Fixed in 4.9.1", but the change will be in 4.9.0 unless it is reverted in branch.

Additionally, I thought plasma uses kdelibs coding style. If that's the case, there need to be braces even in single-statement if/else.

- Christoph Feck


On June 28, 2012, 7:03 p.m., Greg T wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105374/
> -----------------------------------------------------------
> 
> (Updated June 28, 2012, 7:03 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> libtaskmanager can't remove those ("browser", "preferred://browser, , , ") entrys, so I moved them to the Items list. I don't know why they were separated in the first place. That's why I'm opening this review request.
> 
> 
> This addresses bug 278724.
>     http://bugs.kde.org/show_bug.cgi?id=278724
> 
> 
> Diffs
> -----
> 
>   libs/taskmanager/groupmanager.cpp 5ca0159 
> 
> Diff: http://git.reviewboard.kde.org/r/105374/diff/
> 
> 
> Testing
> -------
> 
> no regressions noted.
> 
> 
> Thanks,
> 
> Greg T
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120726/e877509f/attachment.html>