[PATCH wayland-protocols v4 4/7] xdg-shell: Improve error enum formatting some

Quentin Glidic sardemff7+wayland at sardemff7.net
Wed Jul 20 08:54:23 UTC 2016


On 24/06/2016 08:47, Jonas Ådahl wrote:
> The long lines stood out, break them by putting the summary on its own
> line.
>
> Signed-off-by: Jonas Ådahl <jadahl at gmail.com>
> Reviewed-by: Yong Bakos <ybakos at humanoriented.com>
> Reviewed-by: Mike Blumenkrantz <zmike at osg.samsung.com>
> Reviewed-by: Benoit Gschwind <gschwind at gnu-log.net>
> ---

Trivial enough:
Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

Cheers,


>
> No changes since v3.
>
>  unstable/xdg-shell/xdg-shell-unstable-v6.xml | 12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/unstable/xdg-shell/xdg-shell-unstable-v6.xml b/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> index 858c13e..fc41485 100644
> --- a/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> +++ b/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> @@ -37,10 +37,14 @@
>
>      <enum name="error">
>        <entry name="role" value="0" summary="given wl_surface has another role"/>
> -      <entry name="defunct_surfaces" value="1" summary="xdg_shell was destroyed before children"/>
> -      <entry name="not_the_topmost_popup" value="2" summary="the client tried to map or destroy a non-topmost popup"/>
> -      <entry name="invalid_popup_parent" value="3" summary="the client specified an invalid popup parent surface"/>
> -      <entry name="invalid_surface_state" value="4" summary="the client provided an invalid surface state"/>
> +      <entry name="defunct_surfaces" value="1"
> +	     summary="xdg_shell was destroyed before children"/>
> +      <entry name="not_the_topmost_popup" value="2"
> +	     summary="the client tried to map or destroy a non-topmost popup"/>
> +      <entry name="invalid_popup_parent" value="3"
> +	     summary="the client specified an invalid popup parent surface"/>
> +      <entry name="invalid_surface_state" value="4"
> +	     summary="the client provided an invalid surface state"/>
>      </enum>
>
>      <request name="destroy" type="destructor">
>


-- 

Quentin “Sardem FF7” Glidic


More information about the wayland-devel mailing list