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/#review16586
-----------------------------------------------------------


This review has been submitted with commit accddc18bf85850e001c63cfb1d083f4bd9c4b89 by Jekyll Wu to branch KDE/4.9.

- 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/9d91246c/attachment-0001.html>