+ // We will draw our own cursor.
+ add_init_callback([this](){ the_cursor()->hide(); });
This isn't great, as the mir cursor object is still being created. Can we replace Mir's implementation with our own one? -- not a blocker on this MR, can consider this later.
++ src/platforms/mirserver/mirsingleton.cpp
+qtmir::Mir::~Mir()
+{
+ m_instance = nullptr;
+}
You're not deleting what you potentially "new"ed. QScopedPointer helps prevent such accidents...
Ok ignore my relativeMovement thing. I'll not object to the usual terminology.
However I fear I need to ask you to rebase on top of /code.launchpad .net/~gerboland /qtmir/ multimonitor/ +merge/ 269906
https:/
as there's a few conflicts, and the concept of multiple qwindows does make this more complex.
+ // We will draw our own cursor. callback( [this]( ){ the_cursor( )->hide( ); });
+ add_init_
This isn't great, as the mir cursor object is still being created. Can we replace Mir's implementation with our own one? -- not a blocker on this MR, can consider this later.
++ src/platforms/ mirserver/ mirsingleton. cpp
+qtmir::Mir::~Mir()
+{
+ m_instance = nullptr;
+}
You're not deleting what you potentially "new"ed. QScopedPointer helps prevent such accidents...
+++ src/platforms/ mirserver/ mirsingleton. h doc.qt. io/qt-5/ qobject. html#Q_ DISABLE_ COPY
+private:
+ Mir();
maybe http://
Using this Mir singleton to save the cursorName will do fine for now, but I'm wary of it being a dumping ground for lots of little things.