[Development] Header diff for QtQml
Alan Alpert
416365416c at gmail.com
Thu Jun 27 06:30:17 CEST 2013
On Wed, Jun 26, 2013 at 8:10 PM, Thiago Macieira
<thiago.macieira at intel.com> wrote:
> 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).
https://codereview.qt-project.org/#change,59948 implements the latter,
for your perusal.
--
Alan Alpert
More information about the Development
mailing list