From kde-pim  Sun Jan 24 10:05:07 2010
From: Ingo =?iso-8859-1?q?Kl=F6cker?= <kloecker () kde ! org>
Date: Sun, 24 Jan 2010 10:05:07 +0000
To: kde-pim
Subject: Re: [Kde-pim]
Message-Id: <201001241105.12514 () thufir ! ingo-kloecker ! de>
X-MARC-Message: https://marc.info/?l=kde-pim&m=126432756632340
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--===============1228837441=="

--===============1228837441==
Content-type: multipart/signed; boundary=nextPart1753033.GFEHo0K2Lx;
 protocol="application/pgp-signature"; micalg=pgp-sha1
Content-transfer-encoding: 7bit

--nextPart1753033.GFEHo0K2Lx
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

On Saturday 23 January 2010, John Layt wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2709/
> -----------------------------------------------------------
>
> Review request for KDE PIM.
>
>
> Summary
> -------
>
> This change implements phase 1 of the overhaul of KHolidays.  It
> cleans up the file structures, adds some new public api, and adds a
> new parser which supports non-Gregorian calendar systems.  The
> library still uses the old parser for now, but will be switched in
> the next phase.
>
> The holidays.h/.cpp files contained both the Holiday and
> HolidayRegion classes.  These needed to be split to prevent circular
> includes, however it is backwards-compatible as apps still only need
> to include holidays.h.  The following files were created using svn
> copy:
>
>     kholidays/holiday_p.h   // HolidayPrivate class
>     kholidays/holiday.h     // Holiday class
>     kholidays/holiday.cpp
>     kholidays/holidays.h    // Now only HolidayRegion class
>     kholidays/holidays.cpp
>
> (Perhaps also move HolidayRegion into holidayregion.h/.cpp with a
> placeholder holiday.h just including the other headers?

Yes, please. It's unnecessarily confusing and annoying if the name of a=20
header file does not match the class name.


Regards,
Ingo

--nextPart1753033.GFEHo0K2Lx
Content-Type: application/pgp-signature; name=signature.asc 
Content-Description: This is a digitally signed message part.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)

iEYEABECAAYFAktcG1gACgkQGnR+RTDgudgVBACgmplg7KZuLyAiczU2z0OKPCu/
XVMAoLP7m8daeseMElgv2f5Aq6VmDahq
=bjEi
-----END PGP SIGNATURE-----

--nextPart1753033.GFEHo0K2Lx--

--===============1228837441==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
--===============1228837441==--