[PATCH] kdelibs/kdeui/kdatepicker.cpp

Aaron J. Seigo aseigo at kde.org
Thu Nov 8 01:30:14 GMT 2007

On Wednesday 07 November 2007, John Layt wrote:
> OK to commit?

the code looks right upon careful reading of it[1], but i'm far from familiar 
with the internals of this class. are there unit tests for these issues that 
can go along with the fixes to both prevent regressions as well as prove 
these things work? =)

[1] took a moment for me to see that weekYear was getting passed in by 
reference to a method that is in an i18n statement. =) might be worth a 
commentin the code there noting the dual purpose of the next line? *shrug*

Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071107/db4513d9/attachment.sig>

More information about the kde-core-devel mailing list