broulik added inline comments.

INLINE COMMENTS
> screenpool.cpp:188
> +
> +    const xcb_query_extension_reply_t* reply = 
> xcb_get_extension_data(QX11Info::connection(), &xcb_randr_id);
> +

Is that call to X really neccessary everytime? It's called constantly for all 
kinds of events. Also at least cache the QX11Info::connection() ...

REPOSITORY
  R120 Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, sebas, davidedmundson, #plasma
Cc: broulik, pmuralidharan, plasma-devel, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, andreaska, sebas

Reply via email to