[PATCH] Documentation error for KJanusWidget

Waldo Bastian bastian at kde.org
Tue Feb 25 14:50:51 GMT 2003


On Sunday 23 February 2003 21:55, Ravikiran Rajagopal wrote:
> Hello,
>
> > I think the correct fix is to make sure that the index does indeed not
> > change.
>
> I was hoping someone would say that ;-) Please see the attached patches.
> The patches do the following:
>   * Define a private class for binary compatibility
>   * Make sure documentation of pageIndex() holds true
>   * Remove obsolete code (not required since Qt 3)
>   * Add two convenience methods for flexibility
>
> Please review the patches for correctness and binary compatibility. If they
> patches are ok, please commit as I do not have CVS access.

Looks good. Now that you also have CVS access, do you commit it yourself to 
both branches or do you want me to commit it to the 3.1 branch?

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





More information about the kde-core-devel mailing list