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