From 72ce4487f843126026dad6fc0ebe369768c5f22a Mon Sep 17 00:00:00 2001
From: Emmanuele Bassi <ebassi@openedhand.com>
Date: Wed, 7 Nov 2007 16:40:30 +0000
Subject: [PATCH] 2007-11-07  Emmanuele Bassi  <ebassi@openedhand.com>

	* clutter/clutter-actor.[ch]: Fix the declaration of the
	class handlers for the ::captured-event, ::enter-event and
	::leave-event signals; the function signatures were missing
	the return value the signals marshallers were expecting, and
	their names weren't respecting the convention for event-related
	signals.
---
 ChangeLog               |  9 +++++++++
 clutter/clutter-actor.c | 17 +++++++++--------
 clutter/clutter-actor.h | 42 ++++++++++++++++++++---------------------
 3 files changed, 39 insertions(+), 29 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index d78147d3e..ebb8a0a9d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2007-11-07  Emmanuele Bassi  <ebassi@openedhand.com>
+
+	* clutter/clutter-actor.[ch]: Fix the declaration of the
+	class handlers for the ::captured-event, ::enter-event and
+	::leave-event signals; the function signatures were missing
+	the return value the signals marshallers were expecting, and
+	their names weren't respecting the convention for event-related
+	signals.
+
 2007-11-06  Tomas Frydrych  <tf@o-hand.com>
 
         * clutter/clutter-fixed.h:
diff --git a/clutter/clutter-actor.c b/clutter/clutter-actor.c
index f233fea63..bfe365265 100644
--- a/clutter/clutter-actor.c
+++ b/clutter/clutter-actor.c
@@ -129,19 +129,20 @@ enum
   HIDE,
   DESTROY,
   PARENT_SET,
+  FOCUS_IN,
+  FOCUS_OUT,
 
   EVENT,
-  EVENT_CAPTURED,
+  CAPTURED_EVENT,
   BUTTON_PRESS_EVENT,
   BUTTON_RELEASE_EVENT,
   SCROLL_EVENT,
   KEY_PRESS_EVENT,
   KEY_RELEASE_EVENT,
   MOTION_EVENT,
-  FOCUS_IN,
-  FOCUS_OUT,
   ENTER_EVENT,
   LEAVE_EVENT,
+
   LAST_SIGNAL
 };
 
@@ -1538,7 +1539,7 @@ clutter_actor_class_init (ClutterActorClass *klass)
     g_signal_new ("enter-event",
 		  G_TYPE_FROM_CLASS (object_class),
 		  G_SIGNAL_RUN_LAST,
-		  G_STRUCT_OFFSET (ClutterActorClass, enter),
+		  G_STRUCT_OFFSET (ClutterActorClass, enter_event),
 		  _clutter_boolean_handled_accumulator, NULL,
 		  clutter_marshal_BOOLEAN__BOXED,
 		  G_TYPE_BOOLEAN, 1,
@@ -1556,7 +1557,7 @@ clutter_actor_class_init (ClutterActorClass *klass)
     g_signal_new ("leave-event",
 		  G_TYPE_FROM_CLASS (object_class),
 		  G_SIGNAL_RUN_LAST,
-		  G_STRUCT_OFFSET (ClutterActorClass, leave),
+		  G_STRUCT_OFFSET (ClutterActorClass, leave_event),
 		  _clutter_boolean_handled_accumulator, NULL,
 		  clutter_marshal_BOOLEAN__BOXED,
 		  G_TYPE_BOOLEAN, 1,
@@ -1571,11 +1572,11 @@ clutter_actor_class_init (ClutterActorClass *klass)
    *
    * Since: 0.6
    */
-  actor_signals[EVENT_CAPTURED] =
+  actor_signals[CAPTURED_EVENT] =
     g_signal_new ("captured-event",
 		  G_TYPE_FROM_CLASS (object_class),
 		  G_SIGNAL_RUN_LAST,
-		  G_STRUCT_OFFSET (ClutterActorClass, captured),
+		  G_STRUCT_OFFSET (ClutterActorClass, captured_event),
 		  _clutter_boolean_handled_accumulator, NULL,
 		  clutter_marshal_BOOLEAN__BOXED,
 		  G_TYPE_BOOLEAN, 1,
@@ -3122,7 +3123,7 @@ clutter_actor_event (ClutterActor *actor,
 
   if (capture)
     {
-      g_signal_emit (actor, actor_signals[EVENT_CAPTURED], 0,
+      g_signal_emit (actor, actor_signals[CAPTURED_EVENT], 0,
 		     event,
                      &retval);
       goto out;
diff --git a/clutter/clutter-actor.h b/clutter/clutter-actor.h
index 61fb3226a..c27a4a8a4 100644
--- a/clutter/clutter-actor.h
+++ b/clutter/clutter-actor.h
@@ -193,28 +193,28 @@ struct _ClutterActorClass
                             const ClutterColor  *color);
 
   /* event signals */
-  gboolean (* event)                (ClutterActor       *actor,
-				     ClutterEvent       *event);
-  gboolean (* button_press_event)   (ClutterActor       *actor,
-				     ClutterButtonEvent *event);
-  gboolean (* button_release_event) (ClutterActor       *actor,
-				     ClutterButtonEvent *event);
-  gboolean (* scroll_event)         (ClutterActor       *actor,
-				     ClutterScrollEvent *event);
-  gboolean (* key_press_event)      (ClutterActor       *actor,
-				     ClutterKeyEvent    *event);
-  gboolean (* key_release_event)    (ClutterActor       *actor,
-				     ClutterKeyEvent    *event);
-  gboolean (* motion_event)         (ClutterActor       *actor,
-				     ClutterMotionEvent *event);
-  void     (* enter)                (ClutterActor         *actor,
-				     ClutterCrossingEvent *event);
-  void     (* leave)                (ClutterActor         *actor,
-				     ClutterCrossingEvent *event);
-  void     (* captured)             (ClutterActor         *actor,
+  gboolean (* event)                (ClutterActor         *actor,
 				     ClutterEvent         *event);
-  void     (* focus_in)             (ClutterActor       *actor);
-  void     (* focus_out)            (ClutterActor       *actor);
+  gboolean (* button_press_event)   (ClutterActor         *actor,
+				     ClutterButtonEvent   *event);
+  gboolean (* button_release_event) (ClutterActor         *actor,
+				     ClutterButtonEvent   *event);
+  gboolean (* scroll_event)         (ClutterActor         *actor,
+				     ClutterScrollEvent   *event);
+  gboolean (* key_press_event)      (ClutterActor         *actor,
+				     ClutterKeyEvent      *event);
+  gboolean (* key_release_event)    (ClutterActor         *actor,
+				     ClutterKeyEvent      *event);
+  gboolean (* motion_event)         (ClutterActor         *actor,
+				     ClutterMotionEvent   *event);
+  gboolean (* enter_event)          (ClutterActor         *actor,
+				     ClutterCrossingEvent *event);
+  gboolean (* leave_event)          (ClutterActor         *actor,
+				     ClutterCrossingEvent *event);
+  gboolean (* captured_event)       (ClutterActor         *actor,
+				     ClutterEvent         *event);
+  void     (* focus_in)             (ClutterActor         *actor);
+  void     (* focus_out)            (ClutterActor         *actor);
 
   /*< private >*/
   /* padding for future expansion */