D6048: [xembedsniproxy] Fix check whether to use XTest

Wolfgang Bauer noreply at phabricator.kde.org
Wed May 31 10:52:28 UTC 2017


wbauer created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Because of C++'s operator precedence, '!' logically negated all_event_masks only instead of the whole expression.
  This resulted in the condition always being false and XTest never being used.
  
  Adding a pair of brackets fixes it.
  
  BUG: 362941
  BUG: 375017

TEST PLAN
  Verifed with debug output that XTest is being used when XCB_EVENT_MASK_BUTTON_PRESS is not set in all_event_masks.
  
  Tested with a couple of applications:
  
  - mplayerc.exe: wine app, worked before, no XTest, still works
  - tvbrowser, jedit, jitsi 2.8: java apps, didn't work before, use XTest now and work
  - ksensors, ksteak, kconnections, klinpopup, ktvschedule: KDE3 apps, worked before, no XTest, still work
  - wammu: python/wxWidgets app, worked before, no XTest, still works
  - grip, avidemux2, hexchat: gtk2 apps, worked before, no XTest, still work
  - audacious, stardict: gtk3 apps, didn't work before (unless GDK_CORE_DEVICE_EVENTS=1 was set), work now using XTest (without GDK_CORE_DEVICE_EVENTS=1)

REPOSITORY
  R120 Plasma Workspace

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

AFFECTED FILES
  xembed-sni-proxy/sniproxy.cpp

To: wbauer, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170531/d91d0fb5/attachment.html>


More information about the Plasma-devel mailing list