[PATCH v3 1/2] Protocol: Add destructor to wl_data_device interface Fix for Bug# 81745

Giulio Camuffo giuliocamuffo at gmail.com
Fri Oct 10 11:30:30 PDT 2014


2014-09-22 12:28 GMT+03:00 Pekka Paalanen <ppaalanen at gmail.com>:
> On Wed, 27 Aug 2014 16:33:25 +0530
> kabeer.khan at samsung.com wrote:
>
>> From: kabeer <kabeer.khan at samsung.com>
>>
>> Signed-off-by: kabeer <kabeer.khan at samsung.com>
>> ---
>>  protocol/wayland.xml |   11 +++++++++--
>>  1 file changed, 9 insertions(+), 2 deletions(-)
>>
>> diff --git a/protocol/wayland.xml b/protocol/wayland.xml
>> index bb457bc..f0248f5 100644
>> --- a/protocol/wayland.xml
>> +++ b/protocol/wayland.xml
>> @@ -517,7 +517,7 @@
>>
>>    </interface>
>>
>> -  <interface name="wl_data_device" version="1">
>> +  <interface name="wl_data_device" version="2">
>>      <description summary="data transfer device">
>>        There is one wl_data_device per seat which can be obtained
>>        from the global wl_data_device_manager singleton.
>> @@ -640,9 +640,16 @@
>>        </description>
>>        <arg name="id" type="object" interface="wl_data_offer" allow-null="true"/>
>>      </event>
>> +
>> +    <!-- Version 2 additions -->
>> +
>> +    <request name="release" type="destructor" since="2">
>> +      <description summary="destroy data device">
>> +       </description>

I would write something here about what happens when calling it.

>> +    </request>
>>    </interface>
>>
>> -  <interface name="wl_data_device_manager" version="1">
>> +  <interface name="wl_data_device_manager" version="2">
>>      <description summary="data transfer interface">
>>        The wl_data_device_manager is a singleton global object that
>>        provides access to inter-client data transfer mechanisms such as
>
> This looks good to me.
> https://bugs.freedesktop.org/show_bug.cgi?id=81745
>
> Anyone else?
>
>
> Thanks,
> pq
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list