[PATCH weston v2 01/14] libweston: untangle weston_compositor_remove_output doc

Armin Krezović krezovic.armin at gmail.com
Mon Jul 24 16:09:12 UTC 2017


On 24.07.2017 17:07, Pekka Paalanen wrote:
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> Trying to make it more readable. Things that happen in the same step are
> kept in the same paragraph.
> 
> v2: talk about "list of enabled outputs"
> 
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>

Hi,

Seeing that all my concerns were addressed, have a

Reviewed-by: Armin Krezović <krezovic.armin at gmail.com>

But I must ask: The series did contain 15 patches last time,
and this one was no 2, so what happened to the first one from
last time?

Cheers,

Armin.

> ---
>   libweston/compositor.c | 26 ++++++++++++++++----------
>   1 file changed, 16 insertions(+), 10 deletions(-)
> 
> diff --git a/libweston/compositor.c b/libweston/compositor.c
> index 2a3074db..65b3de20 100644
> --- a/libweston/compositor.c
> +++ b/libweston/compositor.c
> @@ -4521,19 +4521,25 @@ weston_output_enable_undo(struct weston_output *output)
>   	output->enabled = false;
>   }
>   
> -/** Removes output from compositor's output list
> +/** Removes output from compositor's list of enabled outputs
>    *
>    * \param output The weston_output object that is being removed.
>    *
> - * Presentation feedback is discarded.
> - * Compositor is notified that outputs were changed and
> - * applies the necessary changes.
> - * All views assigned to the weston_output object are
> - * moved to a new output.
> - * Signal is emitted to notify all users of the weston_output
> - * object that the output is being destroyed.
> - * wl_output protocol objects referencing this weston_output
> - * are made inert.
> + * The following happens:
> + *
> + * - The output assignments of all views in the current scenegraph are
> + *   recomputed.
> + *
> + * - Presentation feedback is discarded.
> + *
> + * - Compositor is notified that outputs were changed and
> + *   applies the necessary changes to re-layout outputs.
> + *
> + * - Signal is emitted to notify all users of the weston_output
> + *   object that the output is being destroyed.
> + *
> + * - wl_output protocol objects referencing this weston_output
> + *   are made inert.
>    */
>   static void
>   weston_compositor_remove_output(struct weston_output *output)
> 



More information about the wayland-devel mailing list