[PATCH] KLocale (configurable first day of the week)

Lukáš Tinkl lukas.tinkl at suse.cz
Fri Jul 19 13:49:15 BST 2002


Hello,
here's a patch that makes the first day of the week freely configurable, not 
just Monday or Sunday as previously. It adds two new methods and one new 
config option:

klocale.h:
QString weekStartDay() const;
void setWeekStartDay(const QString & day);

entry.desktop:
[KCM Locale]
...
WeekStartDay=Mon
...

The config option and the methods take a three-letter English name of the week 
day, e.g. 'Sun' if Sunday is the first day of the week (Monday being the 
default as before).

Question: is it safe to remove the private bool member m_weekStartsMonday?

Question 2: shouldn't we move kdebase/l10n to kdelibs? This way KLocale 
wouldn't depend on data supplied by kdebase which seems logical to me.

Please review

-- 
Lukáš Tinkl
KDE developer | SuSE Labs
http://czechia.kde.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: klocale.cpp.diff
Type: text/x-diff
Size: 2244 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20020719/1dedeb15/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: klocale.h.diff
Type: text/x-diff
Size: 4641 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20020719/1dedeb15/attachment-0001.diff>


More information about the kde-core-devel mailing list