KUser windows port and small API changes

Oswald Buddenhagen ossi at kde.org
Mon Sep 24 18:18:54 BST 2007


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 homephone.
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*.

-- 
Hi! I'm a .signature virus! Copy me into your ~/.signature, please!
--
Chaos, panic, and disorder - my work here is done.




More information about the kde-core-devel mailing list