D8050: Expose wl_display_set_global_filter as a virtual method

Martin Flöser noreply at phabricator.kde.org
Fri Sep 29 15:59:29 UTC 2017


graesslin added inline comments.

INLINE COMMENTS

> davidedmundson wrote in filtered_display.cpp:41
> I tried.
> 
> modding getConnection means modding modding ClientConnection constructor, modding that means modding the Private ctor
> 
> that gets into
> 
>   wl_client_add_destroy_listener(c, &listener);
> 
> c is now const, that fails.
> 
> ----
> 
> We have 3 options:
> 
> - a const_cast somewhere
> - Change Display to create ClientConnections when they come in, not on demand, then we can make getconnection const everywhere. (IMHO, maybe worth it, as our current clientConnected signal is rather misleading)
> - return PID in the API here instead of the client.

ouch, then const_cast is clearly better.

REPOSITORY
  R127 KWayland

REVISION DETAIL
  https://phabricator.kde.org/D8050

To: davidedmundson, #plasma
Cc: graesslin, plasma-devel, #frameworks, leezu, ZrenBot, alexeymin, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170929/948f7795/attachment.html>


More information about the Plasma-devel mailing list