New Krazy check: Forbidden Qt classes
Allen Winter
winter at kde.org
Thu Apr 26 15:40:13 BST 2007
On Wednesday 25 April 2007 7:48:14 pm Allen Winter wrote:
> On Wednesday 25 April 2007 7:19:06 pm David Faure wrote:
> > On Wednesday 25 April 2007, Kevin Ottens wrote:
> > > IIRC a QUrl instead of KUrl in the public API could make sense, but I don't
> > > remember the exact rationale.
> >
> > I strongly disagree. KDE code should use KUrl.
> >
[snip]
> >
>
> Then I should add QUrl to the "prohibited Qt classes" list??
>
> BTW: the check only looks in *.cpp files.
> Which reminds me... it's ok for kdialog.cpp to use QDialog. whoopsie
>
>
OK, the checker will now look for QUrl.
And I fixed some problems.
Are there any other "prohibited" Qt classes? QMessageBox?
And I could probably make a "prohibited" KDE classes checker
that looks for KAction in the public API. Any other KDE classes
that should be avoided?
-Allen
More information about the kde-core-devel
mailing list