Review Request 107696: kcalendarsystem: Simplify weekday naming a lot.

[David Faure]

> On May 6, 2013, 11:40 a.m., Kevin Ottens wrote:
> > Jon, AFAICT this patch hasn't been pushed and it's been way more than a week. Any reason why you didn't push after all?

Looks good to me too. Jon, please push your commits (we can't apply this one since you have other git commits to push with it, like moving stuff to kcalendarsystemweek.cpp, you said....


- David


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


On Jan. 19, 2013, 12:41 p.m., Jon Severinsson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107696/
> -----------------------------------------------------------
> 
> (Updated Jan. 19, 2013, 12:41 p.m.)
> 
> 
> Review request for KDE Frameworks, David Faure and John Layt.
> 
> 
> Description
> -------
> 
> Weekday names depend on *language*, not calendar system. The usual ki18nc can handle it just fine, so consolidate it all to a single function.
> 
> Note: Before this patch I'm moving some functions to a new file "kcalendarsystemweek.cpp" in order move to out of the way of regular calendar system code. That commit changes no code whatsoever, so don't really need a review.
> 
> 
> Diffs
> -----
> 
>   kdecore/date/kcalendarsystem.h c121bb6 
>   kdecore/date/kcalendarsystem.cpp 35877b7 
>   kdecore/date/kcalendarsystemcoptic.cpp 86d35a9 
>   kdecore/date/kcalendarsystemcoptic_p.h 5287271 
>   kdecore/date/kcalendarsystemcopticprivate_p.h 8ceb5ad 
>   kdecore/date/kcalendarsystemethiopian.cpp 9bfc41f 
>   kdecore/date/kcalendarsystemethiopian_p.h 4f8119e 
>   kdecore/date/kcalendarsystemgregorian.cpp 18b7aef 
>   kdecore/date/kcalendarsystemgregorian_p.h 8890643 
>   kdecore/date/kcalendarsystemgregorianprivate_p.h 1208fe9 
>   kdecore/date/kcalendarsystemhebrew.cpp 660cb6d 
>   kdecore/date/kcalendarsystemhebrew_p.h 132e21f 
>   kdecore/date/kcalendarsystemindiannational.cpp 7c4fe03 
>   kdecore/date/kcalendarsystemindiannational_p.h 499d2fa 
>   kdecore/date/kcalendarsystemislamiccivil.cpp abca3b5 
>   kdecore/date/kcalendarsystemislamiccivil_p.h 6d4682f 
>   kdecore/date/kcalendarsystemjalali.cpp 331f2b3 
>   kdecore/date/kcalendarsystemjalali_p.h c88d304 
>   kdecore/date/kcalendarsystemjapanese.cpp d8a870b 
>   kdecore/date/kcalendarsystemjapanese_p.h 09bb4f4 
>   kdecore/date/kcalendarsystemjulian.cpp d1b5864 
>   kdecore/date/kcalendarsystemjulian_p.h 5940c27 
>   kdecore/date/kcalendarsystemminguo.cpp 165c86f 
>   kdecore/date/kcalendarsystemminguo_p.h 0a3b5d7 
>   kdecore/date/kcalendarsystemprivate_p.h 43b486f 
>   kdecore/date/kcalendarsystemweek.cpp PRE-CREATION 
>   kdecore/tests/kcalendartest.h ea5b836 
>   kdecore/tests/kcalendartest.cpp 5c4d438 
> 
> Diff: http://git.reviewboard.kde.org/r/107696/diff/
> 
> 
> Testing
> -------
> 
> `make test` shows no regressions
> 
> 
> Thanks,
> 
> Jon Severinsson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130611/8d152e77/attachment-0001.html>