On 18/11/2016 12:29, Michael Zanetti wrote: >> +Window::Window(int id) >> >+ : QObject(nullptr) >> >+ , m_id(id) >> >+{ >> >+ DEBUG_MSG << "()"; >> >+ QQmlEngine::setObjectOwnership(this, QQmlEngine::CppOwnership); > is there a reason why you can't use QObject's parent instead?
Laziness. Done.
> > Would make sense in any case to parent the Window objects properly IMO > Yes, that's a nice safety net.
« Back to merge proposal
On 18/11/2016 12:29, Michael Zanetti wrote: :setObjectOwner ship(this, QQmlEngine: :CppOwnership) ;
>> +Window::Window(int id)
>> >+ : QObject(nullptr)
>> >+ , m_id(id)
>> >+{
>> >+ DEBUG_MSG << "()";
>> >+ QQmlEngine:
> is there a reason why you can't use QObject's parent instead?
Laziness. Done.
>
> Would make sense in any case to parent the Window objects properly IMO
>
Yes, that's a nice safety net.