Review Request: Make sure kmenuedit use predictable order for showing "Name" and "Description" between various invocations

[Commit Hook]

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


This review has been submitted with commit 440c1a8b2f3332dcd184a3d463c732f136c4b7b8 by Jekyll Wu to branch master.

- Commit Hook


On July 28, 2012, 1:11 p.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105769/
> -----------------------------------------------------------
> 
> (Updated July 28, 2012, 1:11 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> The cause is simple: two flags m_detailedMenuEntries and m_detailedEntriesNamesFirst are never explicitly initialized. So their values are random, and the final results are unpredictable. 
> 
> The patch simply initializes both as true, since that seems to be the default behavior of kickoff.
> 
> 
> The git history shows those two flags seem to be something left over from the KDE3 era. Maybe they should just be removed from the code. However, I'm not familiar with the code and the history, so this simple patch should be safe choice.
> 
> 
> This addresses bug 304177.
>     http://bugs.kde.org/show_bug.cgi?id=304177
> 
> 
> Diffs
> -----
> 
>   kmenuedit/treeview.cpp 54feff7 
> 
> Diff: http://git.reviewboard.kde.org/r/105769/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

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