Looks mostly fine.
I'm a bit iffy about adding the Q_OBJECT macro to classes that don't require it, but am not going to block on it.
Not tested, will wait for silo for that
« Back to merge proposal
Looks mostly fine.
I'm a bit iffy about adding the Q_OBJECT macro to classes that don't require it, but am not going to block on it.
Not tested, will wait for silo for that