<HTML><HEAD><TITLE>Samsung Enterprise Portal mySingle</TITLE>
<META content=IE=5 http-equiv=X-UA-Compatible>
<META content="text/html; charset=windows-1252" http-equiv=Content-Type>
<STYLE id=mysingle_style type=text/css>P {
        MARGIN-BOTTOM: 5px; FONT-SIZE: 9pt; FONT-FAMILY: Arial, arial; MARGIN-TOP: 5px
}
TD {
        MARGIN-BOTTOM: 5px; FONT-SIZE: 9pt; FONT-FAMILY: Arial, arial; MARGIN-TOP: 5px
}
LI {
        MARGIN-BOTTOM: 5px; FONT-SIZE: 9pt; FONT-FAMILY: Arial, arial; MARGIN-TOP: 5px
}
BODY {
        FONT-SIZE: 9pt; FONT-FAMILY: Arial, arial; MARGIN: 10px; LINE-HEIGHT: 1.4
}
</STYLE>

<META name=GENERATOR content=ActiveSquare></HEAD>
<BODY>
<P>Hi</P>
<P>Thank You Pekka for review comments. I will bump the revision on wl_data_device_manager and submit another patch.</P>
<P>With Thanks and Regards</P>
<P>Kabeer</P>
<P>------- <B>Original Message</B> -------</P>
<P><B>Sender</B> : Pekka Paalanen<ppaalanen@gmail.com></P>
<P><B>Date</B> : Aug 26, 2014 20:46 (GMT+09:00)</P>
<P><B>Title</B> : Re: [PATCH v2 1/2] Protocol: Add destructor to wl_data_device interface Fix for Bug# 81745</P>
<P> </P>On Fri, 22 Aug 2014 19:16:49 +0530<BR>kabeer.khan@samsung.com wrote:<BR><BR>> From: kabeer <KABEER.KHAN@SAMSUNG.COM><BR>> <BR>> Signed-off-by: kabeer <KABEER.KHAN@SAMSUNG.COM><BR>> ---<BR>>  protocol/wayland.xml |    9 ++++++++-<BR>>  1 file changed, 8 insertions(+), 1 deletion(-)<BR>> <BR>> diff --git a/protocol/wayland.xml b/protocol/wayland.xml<BR>> index bb457bc..1d3b24f 100644<BR>> --- a/protocol/wayland.xml<BR>> +++ b/protocol/wayland.xml<BR>> @@ -517,7 +517,7 @@<BR>>  <BR>>    </INTERFACE><BR>>  <BR>> -  <INTERFACE name="wl_data_device" version="1"><BR>> +  <INTERFACE name="wl_data_device" version="2"><BR>>      <DESCRIPTION summary="data transfer device"><BR>>        There is one wl_data_device per seat which can be obtained<BR>>        from the global wl_data_device_manager singleton.<BR>> @@ -640,6 +640,13 @@<BR>>        </DESCRIPTION><BR>>        <ARG name="id" type="object" interface="wl_data_offer" allow-null="true" /><BR>>      <BR>> +    <BR>> +    <!-- Version 2 additions --><BR>> +    <BR>> +    <REQUEST name="release" type="destructor" since="2"><BR>> +      <DESCRIPTION summary="destroy data device"><BR>> +   </DESCRIPTION><BR>> +    </REQUEST><BR>>    </INTERFACE><BR>>  <BR>>    <INTERFACE name="wl_data_device_manager" version="1"><BR><BR>Hi,<BR><BR>this is better, just one more thing. When you bump revision here, you<BR>also need to bump the revision on wl_data_device_manager, because that<BR>is the interface that creates wl_data_device objects. This applies<BR>recursively, but wl_data_device_manager is a global, so there is no<BR>interface above it to bump.<BR><BR>Since we are in the alpha phase, I will need another review on the<BR>final series, too. Otherwise this will need to wait until 1.7<BR>development cycle begins.<BR><BR><BR>Thanks,<BR>pq<BR>
<P> </P><!--SP:kabeer.khan--><!--kabeer.khan:EP-->
<P> </P>
<TABLE id=confidentialsignimg>
<TBODY>
<TR>
<TD NAMO_LOCK>
<P><IMG border=0 src="cid:BGFC2LL5XOK0@namo.co.kr"></P></TD></TR></TBODY></TABLE></BODY></HTML><img src='http://ext.samsung.net/mailcheck/SeenTimeChecker?do=c812ce1519221fc6acdea20496c18eb24bcb33aec95e8910ee60ab6642a0204c68faa76fc6d0059275e86b19dccefbe8abdf0e51799c0dfb021b7a989945d95349e5ff3dfdc8681d76f80bf81d31c863cf878f9a26ce15a0' border=0 width=0 height=0 style='display:none'>