[PATCH] Documentation for the prepare_lock_surface event description, is incorrect

[Giulio Camuffo]

2015-07-09 20:12 GMT+03:00 Jasper St. Pierre <jstpierre at mecheye.net>:
> The shell is a client. Both are technically correct, but I'm fine with
> this change if it makes things easier to read. We do use the terms
> "shell" and "client" way too much. :)

Imho this makes it a bit harder to read, as shell can mean either the
client or the shell code in the compositor, while client is just that.

>
> On Thu, Jul 9, 2015 at 10:07 AM, Christopher Michael
> <cpmichael at osg.samsung.com> wrote:
>> Documentation for the prepare_lock_surface event description
>>  is incorrect as it says to "tell the client to create..." however it is
>>  actually the shell which creates the lock surface.
>>
>>
>> Signed-off-by: Chris Michael <cp.michael at samsung.com>
>> ---
>>  protocol/desktop-shell.xml | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/protocol/desktop-shell.xml b/protocol/desktop-shell.xml
>> index fb0b748..147e50d 100644
>> --- a/protocol/desktop-shell.xml
>> +++ b/protocol/desktop-shell.xml
>> @@ -43,7 +43,7 @@
>>      </event>
>>
>>      <event name="prepare_lock_surface">
>> -      <description summary="tell the client to create, set the lock
>> surface">
>> +      <description summary="tell the shell to create, set the lock
>> surface">
>>         Tell the shell we want it to create and set the lock surface, which
>> is
>>         a GUI asking the user to unlock the screen. The lock surface is
>>         announced with 'set_lock_surface'. Whether or not the shell actually
>> --
>> 2.4.4
>> _______________________________________________
>> wayland-devel mailing list
>> wayland-devel at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
>
>
> --
>   Jasper
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel