Merge lp:~jpakkane/libgrip/unbreaking into lp:libgrip

Proposed by Jussi Pakkanen
Status: Merged
Merged at revision: 47
Proposed branch: lp:~jpakkane/libgrip/unbreaking
Merge into: lp:libgrip
Diff against target: 65 lines (+5/-7)
3 files modified
Makefile.am (+0/-3)
examples/rectangle-mover/gesture.c (+4/-4)
src/gripgesturemanager.h (+1/-0)
To merge this branch: bzr merge lp:~jpakkane/libgrip/unbreaking
Reviewer Review Type Date Requested Status
Stephen M. Webb (community) Approve
Review via email: mp+65160@code.launchpad.net

Description of the change

Building works and produces working binaries.

To post a comment you must log in.
Revision history for this message
Stephen M. Webb (bregma) wrote :

I hate to be picky, but could you update the documented parameters to grip_gesture_manager_register_window() (in gripgestiremanager.h) in this change to match the actual parameters? I had to look twice to figure out what was going on.

Just that and I think this is good to go.

review: Needs Fixing
Revision history for this message
Jussi Pakkanen (jpakkane) wrote :

Is it OK now?

lp:~jpakkane/libgrip/unbreaking updated
49. By Jussi Pakkanen

Fixed documentation for register_window.

Revision history for this message
Stephen M. Webb (bregma) wrote :

Absolutely.

review: Approve
Revision history for this message
Chase Douglas (chasedouglas) wrote :

Thanks Jussi! I thought stuff was buildable, but I couldn't test when I merged stuff last week while I was at the Qt conference. Sorry for messing things up!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Makefile.am'
2--- Makefile.am 2011-06-14 17:39:10 +0000
3+++ Makefile.am 2011-06-20 13:07:28 +0000
4@@ -1,8 +1,5 @@
5 SUBDIRS = src bindings examples/rectangle-mover
6
7-libgrip.pc: libgrip.pc
8- cp -f libgrip.pc libgrip.pc
9-
10 pkgconfigdir = $(libdir)/pkgconfig
11 pkgconfig_DATA = libgrip.pc
12
13
14=== modified file 'examples/rectangle-mover/gesture.c'
15--- examples/rectangle-mover/gesture.c 2011-06-17 11:37:17 +0000
16+++ examples/rectangle-mover/gesture.c 2011-06-20 13:07:28 +0000
17@@ -116,32 +116,32 @@
18 grip_gesture_manager_register_window (manager,
19 da,
20 GRIP_GESTURE_PINCH,
21+ GRIP_DEVICE_ALL,
22 2,
23- GRIP_DEVICE_ALL,
24 gesture_callback,
25 NULL, NULL);
26
27 grip_gesture_manager_register_window (manager,
28 da,
29 GRIP_GESTURE_ROTATE,
30+ GRIP_DEVICE_ALL,
31 2,
32- GRIP_DEVICE_ALL,
33 gesture_callback,
34 NULL, NULL);
35
36 grip_gesture_manager_register_window (manager,
37 da,
38 GRIP_GESTURE_DRAG,
39+ GRIP_DEVICE_ALL,
40 2,
41- GRIP_DEVICE_ALL,
42 gesture_callback,
43 NULL, NULL);
44
45 grip_gesture_manager_register_window (manager,
46 da,
47 GRIP_GESTURE_TAP,
48+ GRIP_DEVICE_ALL,
49 2,
50- GRIP_DEVICE_ALL,
51 gesture_callback,
52 NULL, NULL);
53 }
54
55=== modified file 'src/gripgesturemanager.h'
56--- src/gripgesturemanager.h 2011-06-17 11:37:17 +0000
57+++ src/gripgesturemanager.h 2011-06-20 13:07:28 +0000
58@@ -209,6 +209,7 @@
59 * @manager: self
60 * @widget: A #GtkWidget on which to register the gesture
61 * @gesture_type: Gesture type
62+ * @device_type: Type of the device producing the gesture.
63 * @touch_points: Number of touch points
64 * @callback: Callback
65 * @user_data: (transfer none): User data

Subscribers

People subscribed via source and target branches