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