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

On Monday 24 September 2007 19:18:54 Oswald Buddenhagen wrote:
> On Sun, Sep 23, 2007 at 01:40:59AM +0200, Bernhard Loos wrote:
> > +++ kuser.cpp	(working copy)
> > +  /**
> > +   * Retruns an extended property or an invalid
>
> spelling
>
> > +   * QVariant if the property is not set.
> > +   *
> > +   * currently supported properties: roomnumer, workphone and homephone
> > +   *
>
> "Supported properties on UNIX: @c roomnumer, @c workphone and @c homephon=
e.
> Under Windows, no extended properties are supported currently."
>
> > +   * \return a QVariant with value of the property
>
> please use @ consistently.
> actually, i prefer to document the "failure" case here and take out the
> "or invalid", etc. from the description above. oh, well ...
>
> > +   */
> > +  QVariant extendedProperty(const QByteArray &which) const ;
>
> stray space.
>
> other than that, the patch is pre-approved by me and will go in today
> unless somebody cries out *now*.

I suggest to use a type-safe (and faster) enum instead of a string as key. =
I=20
also suggest "extra" as prefix instead of "extended". Or maybe=20
just "property".

	int room =3D user.extendedProperty("roomnumber").toInt();

vs.

	int room =3D user.property(KUser::RoomNumber).toInt();



Simon

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

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQBG9/nZWXvMThJCpvIRAhm3AJ43nThgr51YfL+2pxewtkMPvbKfOwCfTd4b
griWiDf8nMpKXGcDbJn3Zmc=
=a2Dz
-----END PGP SIGNATURE-----

--nextPart1990418.s5ACaZPFK5--