[PATCH v2 02/17] text: Have only one text_model_factory

Jan Arne Petersen jpetersen at openismus.com
Sun Sep 9 14:08:31 PDT 2012


From: Philipp Brüschweiler <blei42 at gmail.com>

Have only one text_model_factory instead of one per seat.

This commit also introduces destruction of an input method when the
corresponding seat is removed.

Signed-off-by: Jan Arne Petersen <jpetersen at openismus.com>
---
 src/compositor.c   |  1 +
 src/compositor.h   |  3 +++
 src/text-backend.c | 75 +++++++++++++++++++++++++++++++++++++++---------------
 3 files changed, 58 insertions(+), 21 deletions(-)

diff --git a/src/compositor.c b/src/compositor.c
index 2b963f5..f76d875 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -3700,6 +3700,7 @@ weston_compositor_init(struct weston_compositor *ec,
 
 	screenshooter_create(ec);
 	text_cursor_position_notifier_create(ec);
+	text_model_factory_create(ec);
 
 	wl_data_device_manager_init(ec->wl_display);
 
diff --git a/src/compositor.h b/src/compositor.h
index 38c2657..bac9e1e 100644
--- a/src/compositor.h
+++ b/src/compositor.h
@@ -741,6 +741,9 @@ void
 text_cursor_position_notifier_create(struct weston_compositor *ec);
 
 void
+text_model_factory_create(struct weston_compositor *ec);
+
+void
 input_method_create(struct weston_compositor *ec,
 		    struct weston_seat *seat);
 
diff --git a/src/text-backend.c b/src/text-backend.c
index 1854e22..5375f3f 100644
--- a/src/text-backend.c
+++ b/src/text-backend.c
@@ -38,13 +38,19 @@ struct text_model {
 	struct wl_surface *surface;
 };
 
+struct text_model_factory {
+	struct wl_global *text_model_factory_global;
+	struct wl_listener destroy_listener;
+
+	struct weston_compositor *ec;
+};
+
 struct input_method {
 	struct wl_resource *input_method_binding;
 	struct wl_global *input_method_global;
-	struct wl_global *text_model_factory_global;
 	struct wl_listener destroy_listener;
 
-	struct weston_compositor *ec;
+	struct weston_seat *seat;
 	struct text_model *model;
 
 	struct wl_list link;
@@ -184,24 +190,24 @@ static void text_model_factory_create_text_model(struct wl_client *client,
 						 struct wl_resource *resource,
 						 uint32_t id)
 {
-	struct input_method *input_method = resource->data;
+	struct text_model_factory *text_model_factory = resource->data;
 	struct text_model *text_model;
 
 	text_model = calloc(1, sizeof *text_model);
 
-	text_model->resource.destroy = destroy_text_model;
-
 	text_model->resource.object.id = id;
 	text_model->resource.object.interface = &text_model_interface;
 	text_model->resource.object.implementation =
 		(void (**)(void)) &text_model_implementation;
-	text_model->resource.data = text_model;
 
-	text_model->ec = input_method->ec;
+	text_model->resource.data = text_model;
+	text_model->resource.destroy = destroy_text_model;
 
-	wl_client_add_resource(client, &text_model->resource);
+	text_model->ec = text_model_factory->ec;
 
 	wl_list_init(&text_model->input_methods);
+
+	wl_client_add_resource(client, &text_model->resource);
 };
 
 static const struct text_model_factory_interface text_model_factory_implementation = {
@@ -214,13 +220,43 @@ bind_text_model_factory(struct wl_client *client,
 			uint32_t version,
 			uint32_t id)
 {
-	struct input_method *input_method = data;
+	struct text_model_factory *text_model_factory = data;
 
 	/* No checking for duplicate binding necessary.
 	 * No events have to be sent, so we don't need the return value. */
 	wl_client_add_object(client, &text_model_factory_interface,
 			     &text_model_factory_implementation,
-			     id, input_method);
+			     id, text_model_factory);
+}
+
+static void
+text_model_factory_notifier_destroy(struct wl_listener *listener, void *data)
+{
+	struct text_model_factory *text_model_factory =
+		container_of(listener, struct text_model_factory, destroy_listener);
+
+	wl_display_remove_global(text_model_factory->ec->wl_display,
+				 text_model_factory->text_model_factory_global);
+
+	free(text_model_factory);
+}
+
+WL_EXPORT void
+text_model_factory_create(struct weston_compositor *ec)
+{
+	struct text_model_factory *text_model_factory;
+
+	text_model_factory = calloc(1, sizeof *text_model_factory);
+
+	text_model_factory->ec = ec;
+
+	text_model_factory->text_model_factory_global =
+		wl_display_add_global(ec->wl_display,
+				      &text_model_factory_interface,
+				      text_model_factory, bind_text_model_factory);
+
+	text_model_factory->destroy_listener.notify = text_model_factory_notifier_destroy;
+	wl_signal_add(&ec->destroy_signal, &text_model_factory->destroy_listener);
 }
 
 static void
@@ -279,10 +315,12 @@ input_method_notifier_destroy(struct wl_listener *listener, void *data)
 	struct input_method *input_method =
 		container_of(listener, struct input_method, destroy_listener);
 
-	wl_display_remove_global(input_method->ec->wl_display,
+	if (input_method->model)
+		deactivate_text_model(input_method->model, input_method);
+
+	wl_display_remove_global(input_method->seat->compositor->wl_display,
 				 input_method->input_method_global);
-	wl_display_remove_global(input_method->ec->wl_display,
-				 input_method->text_model_factory_global);
+
 	free(input_method);
 }
 
@@ -316,7 +354,7 @@ input_method_init_seat(struct weston_seat *seat)
 	seat->input_method->focus_listener_initialized = 1;
 }
 
-void
+WL_EXPORT void
 input_method_create(struct weston_compositor *ec,
 		    struct weston_seat *seat)
 {
@@ -324,7 +362,7 @@ input_method_create(struct weston_compositor *ec,
 
 	input_method = calloc(1, sizeof *input_method);
 
-	input_method->ec = ec;
+	input_method->seat = seat;
 	input_method->model = NULL;
 	input_method->focus_listener_initialized = 0;
 
@@ -333,13 +371,8 @@ input_method_create(struct weston_compositor *ec,
 				      &input_method_interface,
 				      input_method, bind_input_method);
 
-	input_method->text_model_factory_global =
-		wl_display_add_global(ec->wl_display,
-				      &text_model_factory_interface,
-				      input_method, bind_text_model_factory);
-
 	input_method->destroy_listener.notify = input_method_notifier_destroy;
-	wl_signal_add(&ec->destroy_signal, &input_method->destroy_listener);
+	wl_signal_add(&seat->seat.destroy_signal, &input_method->destroy_listener);
 
 	seat->input_method = input_method;
 }
-- 
1.7.11.4



More information about the wayland-devel mailing list