[PATCH] Documentation error for KJanusWidget

Waldo Bastian bastian at kde.org
Sun Feb 23 18:24:47 GMT 2003


On Sunday 23 February 2003 18:29, Ravikiran Rajagopal wrote:
> Hello,
>   KJanusWidget::pageIndex() has wrong documentation. Aargh, this one cost
> me a whole day! Patch, someone please apply:
>
> --- kjanuswidget.h.orig 2003-02-23 12:17:20.000000000 -0500
> +++ kjanuswidget.h      2003-02-23 12:20:30.000000000 -0500
> @@ -369,8 +369,8 @@
>       * @ref activePageIndex if you need to do some page specific actions
>       * in your code.
>       *
> -     * The returned index will never change so you can safely use this
> -     * function once and save the value.
> +     * The returned index will not change unless a page is deleted. If a
> page +     * is deleted/removed, the indices of all remaining pages may
> change. *
>       * @param widget The widget returned by @ref addPage , @ref
> addVBoxPage ,

I think the correct fix is to make sure that the index does indeed not change.

Cheers,
Waldo
-- 
bastian at kde.org -=|[ SuSE, The Linux Desktop Experts ]|=- bastian at suse.com





More information about the kde-core-devel mailing list