Header cleanup / Link interface (was kdewidgets: fix build)

[Kevin Ottens]

On Friday 19 July 2013 20:11:32 David Faure wrote:
> On Friday 19 July 2013 17:40:26 wojtask w wrote:
> > Before this task I want to cleanup public headers in tier1, tier2 and
> > stagging.
> > I have a question :)
> > 1. What is preferred <QtCore/QString> or <QString>?
> 
> Both work, so it doesn't really matter.
> 
> The qt4 theory was "always use QtCore/QString in public headers so that the
> include path doesn't have to contain QtCore, for crappy build systems" (not
> qmake, not cmake, so stuff like hand-written Makefiles or MSVC projects).
> 
> However the qt4-to-qt5 porting taught us that this created problems
> (QtGui/QLabel became QtWidgets/QLabel == additional porting, for an issue
> that wouldn't happen if we just used <QLabel>).
> 
> So I don't know ;)
> 
> If you want to make it consistent, let's just do like Qt does....
> Oh great -- they do something different: #include <QtCore/qstring.h>
> 
> Adding k-f-d to the CC. What do you guys think? Major search/replace, to
> follow Qt style?

I think we're good with just "#include <QLabel>" (for the porting reasons you 
invoked).

Regards.
-- 
Kévin Ottens, http://ervin.ipsquad.net

Sponsored by BlueSystems and KDAB to work on KDE Frameworks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130722/724ae7cb/attachment.sig>