Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 388528 - Use gdk_device_ungrab() instead of gdk_pointer_ungrab() and gdk_keyboard_ungrab() in GTK3+
Summary: Use gdk_device_ungrab() instead of gdk_pointer_ungrab() and gdk_keyboard_ungr...
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.2   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.3 M3   Edit
Assignee: Arun Thondapu CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 340067
  Show dependency tree
 
Reported: 2012-08-31 09:40 EDT by Anatoly Spektor CLA
Modified: 2012-09-28 08:56 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Anatoly Spektor CLA 2012-08-31 09:40:19 EDT 
This patch replaces gdk_pointer_ungrab() with gdk_device_ungrab() for Gtk3+ and higher:


http://fedorapeople.org/cgit/aspektor/public_git/eclipse.platform.swt.git/commit/?h=gtk3_ungrab_pointer
Comment 1 Anatoly Spektor CLA 2012-08-31 12:32:09 EDT 
Update:

 As there is only one occurrence of deprecated gdk_keyboard_ungrab () and it is fully dependant on gdk_device_ungrab(), I have decided to put both patches in the same branch.

 Thus please refer to the updated links to 2 commits:

 Patch that uses gdk_device_ungrab() instead of gdk_pointer_ungrab(): 

 1. http://fedorapeople.org/cgit/aspektor/public_git/eclipse.platform.swt.git/commit/?h=gtk3_ungrab_pointer&id=f77b4014c8ef4f9962363ed94775e225f0d25865 

  Patch that gets rid of deprecated gdk_keyboard_ungrab () (dependant on the commit above):
 2. http://fedorapeople.org/cgit/aspektor/public_git/eclipse.platform.swt.git/commit/?h=gtk3_ungrab_pointer&id=67a460a42be09d4e50200d710407774fdc30d38a


 If this way is not acceptable and you want me to create separate bug report for gdk_keyboard_ungrab () please let me know, I will do it, I just thought that as second commit depends on first one, it is easier to track them both here.

Regards,

Anatoly