[Development] Header diff for QtQml

Thiago Macieira thiago.macieira at intel.com
Thu Jun 27 05:10:26 CEST 2013


On quarta-feira, 26 de junho de 2013 17.42.54, Alan Alpert wrote:
> > Does it have a reason for the include? For example, does it declare
> > debugging operators for anything? If not, it's probably accidental and
> > can be safely removed.
> 
> It uses qWarning() in the definition of a new inline function (the url
> form of qmlRegisterType). Sounds like it's okay to leave it there,
> although the function could be un-inlined if the include is a problem.

I'd prefer that. Or use the non-streaming version of qWarning(), which doesn't 
require qdebug.h (only qlogging.h, which is included from qglobal.h).

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.qt-project.org/pipermail/development/attachments/20130626/d1f6b353/attachment.sig>


More information about the Development mailing list