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

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

- Commit


On July 28th, 2012, 1:11 p.m., Jekyll Wu wrote:

Review request for Plasma.
By Jekyll Wu.

Updated July 28, 2012, 1:11 p.m.

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.
Bugs: 304177

Diffs

  • kmenuedit/treeview.cpp (54feff7)

View Diff