Re: [PATCH] protocol: deprecate wl_surface.damage

2018-11-04 Thread Derek Foreman
IMHO this is a win.

Reviewed-by: Derek Foreman 

I think we can follow it up by removing surface damage in surface
co-ordinates from weston clients - if we don't want people using it, we
shouldn't be giving then good examples of how.

I also think it's reasonable for compositor writers to take the easy way
out of handling legacy surface damage by just treating any
wl_surface.damage request as full surface damage...

Thanks,
Derek

On 11/4/18 2:38 AM, Simon Ser wrote:
> This commit deprecates wl_surface.damage in favor of wl_surface.damage_buffer.
> 
> Having two requests makes it complicated for the compositor to handle damage,
> making it necessary to transform one into the other's coordinates.
> 
> Moreover, integration with wp_viewporter is tricky.
> 
> Signed-off-by: Simon Ser 
> ---
> 
> Based on the discussion in [1].
> 
> [1]: 
> https://gitlab.freedesktop.org/wayland/weston/merge_requests/41#note_74071
> 
>  protocol/wayland.xml | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/protocol/wayland.xml b/protocol/wayland.xml
> index 141038b..c75ea3a 100644
> --- a/protocol/wayland.xml
> +++ b/protocol/wayland.xml
> @@ -1397,9 +1397,9 @@
>   and clears pending damage. The server will clear the current
>   damage as it repaints the surface.
>  
> - Alternatively, damage can be posted with wl_surface.damage_buffer
> - which uses buffer coordinates instead of surface coordinates,
> - and is probably the preferred and intuitive way of doing this.
> + Note! This request is deprecated and should not be used in new
> + clients. Instead damage can be posted with wl_surface.damage_buffer
> + which uses buffer coordinates instead of surface coordinates.
>
>
>
> 

___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


I would like to learn how this project build Wayland

2018-11-04 Thread Valentino Bocchino
Hi,
I’m Valentino, I come from Italy and I’m 14 years old. I am really interested 
in learning programming languages and I thought that asking the Wayland team 
how they built Wayland could be a great idea. I don’t carry any kind of 
experience, so I probably can’t help to this project, but if I could learn, 
then maybe one day I could become helpful. Of course, it is unlikely that I can 
be physically present, but the technology that we have today enables me to 
learn and help people in other places which revery distant from home.

Have a nice day,
Valentino Bocchino
P.S.: Excuse me for my English, but because it’s my third language I’m trying 
to do my best.
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


[no subject]

2018-11-04 Thread cmailings

___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


[PATCH] protocol: deprecate wl_surface.damage

2018-11-04 Thread Simon Ser
This commit deprecates wl_surface.damage in favor of wl_surface.damage_buffer.

Having two requests makes it complicated for the compositor to handle damage,
making it necessary to transform one into the other's coordinates.

Moreover, integration with wp_viewporter is tricky.

Signed-off-by: Simon Ser 
---

Based on the discussion in [1].

[1]: https://gitlab.freedesktop.org/wayland/weston/merge_requests/41#note_74071

 protocol/wayland.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/protocol/wayland.xml b/protocol/wayland.xml
index 141038b..c75ea3a 100644
--- a/protocol/wayland.xml
+++ b/protocol/wayland.xml
@@ -1397,9 +1397,9 @@
and clears pending damage. The server will clear the current
damage as it repaints the surface.
 
-   Alternatively, damage can be posted with wl_surface.damage_buffer
-   which uses buffer coordinates instead of surface coordinates,
-   and is probably the preferred and intuitive way of doing this.
+   Note! This request is deprecated and should not be used in new
+   clients. Instead damage can be posted with wl_surface.damage_buffer
+   which uses buffer coordinates instead of surface coordinates.
   
   
   
-- 
2.19.1


___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel