https://bugs.kde.org/show_bug.cgi?id=499383

Sune Vuorela <k...@pusling.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |k...@pusling.com

--- Comment #37 from Sune Vuorela <k...@pusling.com> ---
(In reply to François Guerraz from comment #36)
> FWIW my patch works fine for me, so I’m okay, but it’s not a platform I want
> to invest time in learning in any more depth.

I'm not sure how you want feedback to your patch, but given there exists
systems like gerrit to enable others to actually work with you on fixing your
patch, which I as an absolute wayland noob can tell is at least far too big and
is filling me with more 'why he changes this?' rather than 'this makes sense'.

For example, `oldestEnteredScreen()` already checks that screen->screen() is
valid before returning it, so why are you doing it again?

and under which circumstances can a `QWaylandWindow` not have a `window` and
thus the questions keeps piling up, but how do I ask them in a way where you
can answer them when you are refusing to sign up for the systems where we are
able to it easily?

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to