[PATCH] Bug 158444: Pager can't display name of desktop
Rafał Miłecki
zajec5 at gmail.com
Thu Mar 13 14:49:10 CET 2008
2008/3/12, Aaron J. Seigo <aseigo at kde.org>:
> On Wednesday 12 March 2008, Rafał Miłecki wrote:
> > I prepared patch for bug https://bugs.kde.org/show_bug.cgi?id=158444
> >
> > Could someone check it and apply if everything is fine?
>
>
> + int m_displayedText; // 0==desktop number ; 1==desktop name ; 2==none
>
> using an enumeration rather than an integer with magic values that have the be
> documented in this way would make it more readable and maintainable.
Could you give me some tip, how can I use
KConfigGroup cg = config();
cg.writeEntry("displayedText", m_displayedText);
when m_displayedText is an enum? writeEntry doesn't acceps such a
values and doesn't let me read it later.
I asked for help on #kde-devel and someone gave me link
http://websvn.kde.org/*checkout*/trunk/KDE/kdepim/libkdepim/kaccount.cpp
but I can't see anything really usefully here.
--
Rafał Miłecki
More information about the Panel-devel
mailing list