libweston: rename weston_output_destroy() to weston_output_release()

'release' is a more appropriate name because the function does not free
the underlying memory. The main reason for this is that we need the name
weston_output_destroy() for new API that actually will free also the
underlying memory.

Since the function is only used in backends and external backends are
not a thing, this does not cause libweston major version bump, even
though it does change the ABI. There is no way external users could have
successfully used this function.

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Ian Ray <ian.ray@ge.com>
Acked-by Daniel Stone <daniels@collabora.com>

diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c
index a2e6db4..6aca10c 100644 (file)
--- a/libweston/compositor-drm.c
+++ b/libweston/compositor-drm.c
@@ -3272,7 +3272,7 @@ drm_output_destroy(struct weston_output *base)
        if (output->pageflip_timer)
                wl_event_source_remove(output->pageflip_timer);
 
-       weston_output_destroy(&output->base);
+       weston_output_release(&output->base);
 
        drm_property_info_free(output->props_conn, WDRM_CONNECTOR__COUNT);
 

diff --git a/libweston/compositor-fbdev.c b/libweston/compositor-fbdev.c
index fe169ca..1765fa8 100644 (file)
--- a/libweston/compositor-fbdev.c
+++ b/libweston/compositor-fbdev.c
@@ -555,7 +555,7 @@ fbdev_output_destroy(struct weston_output *base)
        fbdev_output_disable_handler(base);
 
        /* Remove the output. */
-       weston_output_destroy(&output->base);
+       weston_output_release(&output->base);
 
        free(output->device);
        free(output);

diff --git a/libweston/compositor-headless.c b/libweston/compositor-headless.c
index ac762c1..64fe6f3 100644 (file)
--- a/libweston/compositor-headless.c
+++ b/libweston/compositor-headless.c
@@ -134,7 +134,7 @@ headless_output_destroy(struct weston_output *base)
        struct headless_output *output = to_headless_output(base);
 
        headless_output_disable(&output->base);
-       weston_output_destroy(&output->base);
+       weston_output_release(&output->base);
 
        free(output);
 }

diff --git a/libweston/compositor-rdp.c b/libweston/compositor-rdp.c
index f2fa0f4..393c118 100644 (file)
--- a/libweston/compositor-rdp.c
+++ b/libweston/compositor-rdp.c
@@ -570,7 +570,7 @@ rdp_output_destroy(struct weston_output *base)
        struct rdp_output *output = to_rdp_output(base);
 
        rdp_output_disable(&output->base);
-       weston_output_destroy(&output->base);
+       weston_output_release(&output->base);
 
        free(output);
 }
@@ -1360,7 +1360,7 @@ rdp_backend_create(struct weston_compositor *compositor,
 err_listener:
        freerdp_listener_free(b->listener);
 err_output:
-       weston_output_destroy(&b->output->base);
+       weston_output_release(&b->output->base);
 err_compositor:
        weston_compositor_shutdown(compositor);
 err_free_strings:

diff --git a/libweston/compositor-wayland.c b/libweston/compositor-wayland.c
index e8639a0..26cfdd5 100644 (file)
--- a/libweston/compositor-wayland.c
+++ b/libweston/compositor-wayland.c
@@ -703,7 +703,7 @@ wayland_output_destroy(struct weston_output *base)
 
        wayland_output_disable(&output->base);
 
-       weston_output_destroy(&output->base);
+       weston_output_release(&output->base);
 
        if (output->frame_cb)
                wl_callback_destroy(output->frame_cb);
@@ -1376,7 +1376,7 @@ wayland_output_create_for_parent_output(struct wayland_backend *b,
        return 0;
 
 out:
-       weston_output_destroy(&output->base);
+       weston_output_release(&output->base);
        free(output->title);
        free(output);
 
@@ -1424,7 +1424,7 @@ wayland_output_create_fullscreen(struct wayland_backend *b)
 err_set_size:
        wayland_backend_destroy_output_surface(output);
 err_surface:
-       weston_output_destroy(&output->base);
+       weston_output_release(&output->base);
        free(output->title);
        free(output);
 

diff --git a/libweston/compositor-x11.c b/libweston/compositor-x11.c
index 9d37142..070f287 100644 (file)
--- a/libweston/compositor-x11.c
+++ b/libweston/compositor-x11.c
@@ -801,7 +801,7 @@ x11_output_destroy(struct weston_output *base)
        struct x11_output *output = to_x11_output(base);
 
        x11_output_disable(&output->base);
-       weston_output_destroy(&output->base);
+       weston_output_release(&output->base);
 
        free(output);
 }

diff --git a/libweston/compositor.c b/libweston/compositor.c
index e647d30..53bbf55 100644 (file)
--- a/libweston/compositor.c
+++ b/libweston/compositor.c
@@ -4750,7 +4750,7 @@ weston_output_init(struct weston_output *output,
  * Also notifies the compositor that an output is pending for
  * configuration.
  *
- * The opposite of this operation is built into weston_output_destroy().
+ * The opposite of this operation is built into weston_output_release().
  *
  * \memberof weston_output
  * \internal
@@ -4931,7 +4931,7 @@ weston_pending_output_coldplug(struct weston_compositor *compositor)
  * \internal
  */
 WL_EXPORT void
-weston_output_destroy(struct weston_output *output)
+weston_output_release(struct weston_output *output)
 {
        output->destroying = 1;
 

diff --git a/libweston/compositor.h b/libweston/compositor.h
index 804d091..8b2d2b0 100644 (file)
--- a/libweston/compositor.h
+++ b/libweston/compositor.h
@@ -1717,7 +1717,7 @@ void
 weston_output_move(struct weston_output *output, int x, int y);
 
 void
-weston_output_destroy(struct weston_output *output);
+weston_output_release(struct weston_output *output);
 void
 weston_output_transform_coordinate(struct weston_output *output,
                                   double device_x, double device_y,