[PATCH] kdelibs/kdeui/kdatepicker.cpp

Aaron J. Seigo aseigo at kde.org
Thu Nov 8 23:21:03 GMT 2007


On Thursday 08 November 2007, John Layt wrote:
> On Thursday 08 November 2007, Aaron J. Seigo wrote:
> > 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*
>
> Reworked patch attached with better comments and clearer workings.

for someone not intimately faimiliar with these things, this patch looks a lot 
clearer =) thanks...

> At the moment, the unit test consists of running
> kdecore/kdeui/tests/kdatepicktest and manually selecting dates, then
> manually comparing to the KDE3 version.  Proper unit tests are planned for
> 4.1.

sweet.

> P.S. D'oh, just remembered about the kdelibs-bugs list...

heh.. well, as long as it gets fixed that's all that matters =)

-- 
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/20071108/1fce73c8/attachment.sig>


More information about the kde-core-devel mailing list