KFileWidget and KDirOperator: request for unobtrusive interface change
Peter Penz
peter.penz at gmx.at
Wed Sep 5 08:57:00 BST 2007
Hi,
during my KDE4 porting for KFileWidget and KDirOperator I faced some
interfaces which should be fixed, but I initially thought to postpone this
for KDE5 as we somewhen have to stop changing interfaces...
Anyway after commiting a comment for KDE 5 Oswald replied:
> > SVN commit 708602 by ppenz:
> >
> > added comment for KDE5 (no typo ;-))
> >
> > --- trunk/KDE/kdelibs/kfile/kfilewidget.h #708601:708602
> > @@ -257,7 +257,8 @@
> > - virtual void setPreviewWidget(const KPreviewWidgetBase *w);
> > + // ### KDE5: remove 'const', as the widget must get layouted and
> > will be deleted + virtual void setPreviewWidget(const
> > KPreviewWidgetBase *w);
>
> that's silly. there is no reason not to make such an unobtrusive change
> next monday.
I'm not aware about any application using a custom preview widget in the file
dialog, so I agree to Oswald that the interface change would be unobtrusive.
Changing the interface would imply to change:
- void KDirOperator::setPreviewWidget(const QWidget *w)
+ void KDirOperator::setPreviewWidget(KPreviewWidgetBase *w)
to prevent nasty casts...
Is it OK doing this change next Monday? I fully understand if the answer
is 'no'...
Thanks!
Peter
More information about the kde-core-devel
mailing list