<div dir="ltr">I will push new patch with minor fix to the strtol function in wayland and move this old patch (after segfault fix) to weston so that it does not end up in libwayland APIs.<div>Consequently I changed its property in patchwork<br><div><br></div><div>BR</div><div>imran</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 29, 2014 at 10:27 AM, Imran Zaman <span dir="ltr"><<a href="mailto:imran.zaman@gmail.com" target="_blank">imran.zaman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Wed, Oct 29, 2014 at 10:09 AM, Giulio Camuffo <span dir="ltr"><<a href="mailto:giuliocamuffo@gmail.com" target="_blank">giuliocamuffo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span>2014-10-29 8:45 GMT+02:00 Imran Zaman <<a href="mailto:imran.zaman@gmail.com" target="_blank">imran.zaman@gmail.com</a>>:<br>
> Daniel!<br>
><br>
> As per your logic, I see wl_list APIs exposed etc, which shouldn't be part<br>
> of libwayland as well.<br>
> similarly, wl_fixed_to_double and wl_array shouldn't be part of the window<br>
> system. Isnt it?<br>
> I can make inline functions if that helps.<br>
<br>
</span>wl_list is used in the server side API, so it's a bit different.<br>
However, I'd agree that it'd be better if it wasn't exposed but we<br>
can't remove it now. wl_fixed is a wayland specific type so all the<br>
wl_fixed_* functions need to be part of the API.<br>
On the other hand wl_strtol would just be a function, there are is no<br>
other API that depends on it.<br>
<span><br>
><br>
> Btw here is an API patch, which has not be reviewed till now.<br>
> <a href="http://lists.freedesktop.org/archives/wayland-devel/2014-October/017833.html" target="_blank">http://lists.freedesktop.org/archives/wayland-devel/2014-October/017833.html</a><br>
<br>
</span>Yes, like there are many other patches waiting for reviews. You need<br>
to have patience, it's not like we are ignoring it. But, if I may add,<br>
the way you are reacting to a comment to this patch doesn't encourage<br>
to review your other ones.<br>
<br>
<br></blockquote></span><div><br>Neither the random/comments to the patch are encouraging :-) e.g. "<span style="font-size:12.7272720336914px;font-family:arial,sans-serif">AOL. We're a window system, not a replacement libc.</span>"</div><div>Its your choice to review it or not; I did not put the API patch link here just because it has not been reviewed. I have lots of patience but Tizen may need something urgent or else we may need to fork wayland/weston in Tizen. I put it in the thread because Daniel said that we did not have any further progress/discussion on that end.</div><div><br></div><div>Anyways I take the patch off, as it does not "sound" like it should be here to save everyone's time. If the time allows, I will remove it from public APIs in addition to one critical bug fix and resubmit the patch.</div><div><div class="h5"><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
--<br>
Giulio<br>
<div><div><br>
><br>
> BR<br>
> imran<br>
><br>
> On Tue, Oct 28, 2014 at 6:36 PM, Daniel Stone <<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>> wrote:<br>
><br>
>> Hi,<br>
>><br>
>> On 28 October 2014 15:40, Imran Zaman <<a href="mailto:imran.zaman@gmail.com" target="_blank">imran.zaman@gmail.com</a>> wrote:<br>
>>><br>
>>> You guys should check the reason why the patch is there rather than<br>
>>> throwing out random thoughts or blunt comments.<br>
>>><br>
>>> I did this patch mainly because weston/wayland has been using<br>
>>> strtol/strtoul functions in number of places with buggy error checks, and<br>
>>> duplicate code everywhere. Weston and wayland go together; so in bigger<br>
>>> picture, its a very useful patch IMO.. I hardly find any patches with proper<br>
>>> tests, but I did it so to make it more effective. But I guess in<br>
>>> wayland/weston community, only maintainers are allowed to push patches<br>
>>> others are strongly discouraged to do so. I guess its better to encourage<br>
>>> people/community for giving helping hand.<br>
>>><br>
>>> Anyways we will now only push patches (including multi-seat support) in<br>
>>> Tizen weston/wayland rather than wasting time in upstreamn weston/wayland as<br>
>>> it seems to be long bureaucratic process to overcome with virtually no<br>
>>> success.<br>
>><br>
>><br>
>> That's not what we've said. No-one said 'don't take the patch'; all we<br>
>> said is 'please don't expose it as part of libwayland-*'s _public_ API'.<br>
>><br>
>> I like the idea of the patch, I like how it's caught a number of buggy<br>
>> spots where we've open-coded the same thing, and I like that it's<br>
>> well-tested. For internal usage, it's great! I just don't want to expose<br>
>> string manipulation functions in the public API of a window system.<br>
>><br>
>> You're right that the test infrastructure is in a pretty bad state.<br>
>> Anything which helps that is more than welcome, and you may have seen a<br>
>> bunch of patches from Derek Foreman (not a maintainer) on this list, which<br>
>> are progressing well and go a long way towards improving our test suite into<br>
>> something that will be really useful day to day. Any further contributions<br>
>> along those lines - from anyone - are totally welcome.<br>
>><br>
>> As for your multiseat patches, the last discussions I remember involved<br>
>> some pretty fundamental disagreements about the whole architecture,<br>
>> particularly input support. I haven't seen any more patches or discussion<br>
>> since the last IRC chat, though.<br>
>><br>
>> Cheers,<br>
>> Daniel<br>
>><br>
><br>
> On Tue, Oct 28, 2014 at 6:36 PM, Daniel Stone <<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> On 28 October 2014 15:40, Imran Zaman <<a href="mailto:imran.zaman@gmail.com" target="_blank">imran.zaman@gmail.com</a>> wrote:<br>
>>><br>
>>> You guys should check the reason why the patch is there rather than<br>
>>> throwing out random thoughts or blunt comments.<br>
>>><br>
>>> I did this patch mainly because weston/wayland has been using<br>
>>> strtol/strtoul functions in number of places with buggy error checks, and<br>
>>> duplicate code everywhere. Weston and wayland go together; so in bigger<br>
>>> picture, its a very useful patch IMO.. I hardly find any patches with proper<br>
>>> tests, but I did it so to make it more effective. But I guess in<br>
>>> wayland/weston community, only maintainers are allowed to push patches<br>
>>> others are strongly discouraged to do so. I guess its better to encourage<br>
>>> people/community for giving helping hand.<br>
>>><br>
>>> Anyways we will now only push patches (including multi-seat support) in<br>
>>> Tizen weston/wayland rather than wasting time in upstreamn weston/wayland as<br>
>>> it seems to be long bureaucratic process to overcome with virtually no<br>
>>> success.<br>
>><br>
>><br>
>> That's not what we've said. No-one said 'don't take the patch'; all we<br>
>> said is 'please don't expose it as part of libwayland-*'s _public_ API'.<br>
>><br>
>> I like the idea of the patch, I like how it's caught a number of buggy<br>
>> spots where we've open-coded the same thing, and I like that it's<br>
>> well-tested. For internal usage, it's great! I just don't want to expose<br>
>> string manipulation functions in the public API of a window system.<br>
>><br>
>> You're right that the test infrastructure is in a pretty bad state.<br>
>> Anything which helps that is more than welcome, and you may have seen a<br>
>> bunch of patches from Derek Foreman (not a maintainer) on this list, which<br>
>> are progressing well and go a long way towards improving our test suite into<br>
>> something that will be really useful day to day. Any further contributions<br>
>> along those lines - from anyone - are totally welcome.<br>
>><br>
>> As for your multiseat patches, the last discussions I remember involved<br>
>> some pretty fundamental disagreements about the whole architecture,<br>
>> particularly input support. I haven't seen any more patches or discussion<br>
>> since the last IRC chat, though.<br>
>><br>
>> Cheers,<br>
>> Daniel<br>
><br>
><br>
</div></div></blockquote></div></div></div><br></div></div>
</blockquote></div><br></div>