<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 9 December 2014 at 17:08, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, 3 Dec 2014 13:05:16 +0100<br>
<span class="">Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>> wrote:<br>
<br>
> On 28 November 2014 at 14:30, Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com">ppaalanen@gmail.com</a>> wrote:<br>
><br>
> > On Fri, 28 Nov 2014 12:18:55 +0100<br>
> > Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>> wrote:<br>
> ><br>
> > > 1) there is nothing like main thread since<br>
> > >    3c7e8bfbb4745315b7bcbf69fa746c3d6718c305 anymore, so remove<br>
> > >    it from documentation and update the doc accordingly.<br>
> > ><br>
> > > 2) use calling 'default queue' instead of 'main queue'. In the code<br>
> > >    we use display->default_queue, so it'll be easier the understand.<br>
> > ><br>
> > > 3) update some obsolete or unprecise pieces of documentation<br>
> > ><br>
> > > Signed-off-by: Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>><br>
> > > ---<br>
> > >  src/wayland-client.c | 80<br>
> > ++++++++++++++++++++++++++++++++--------------------<br>
> > >  src/wayland-client.h | 26 ++++++++---------<br>
> > >  2 files changed, 61 insertions(+), 45 deletions(-)<br>
> > ><br>
> > > diff --git a/src/wayland-client.c b/src/wayland-client.c<br>
> > > index 36380fe..0ab94e9 100644<br>
> > > --- a/src/wayland-client.c<br>
> > > +++ b/src/wayland-client.c<br>
<br>
</span>...<br>
<span class=""><br>
> > > +<br>
> > > + * \note Display queue (for wl_display events like delete_id or so)<br>
> > > + * is dispatched always.<br>
> ><br>
> > I wonder if referring to the display queue is just confusing, because I<br>
> > don't see us really explaining what it is anywhere. And it's not the<br>
> > default queue. In fact, I don't think the users of libwayland would even<br>
> > care about it.<br>
> ><br>
><br>
> True.<br>
><br>
> Hmm, just thinking about it - I should rephrase the thing about checking<br>
> return value of wl_display_dispatch_queue(). Because when<br>
> it dispatched some events in display_queue, then it won't return 0 even<br>
> when it dispatched none of the queue-assigned events, so what I wrote in<br>
> the comment to wl_display_dispatch_queue() is wrong.<br>
><br>
> But to clarify that it can return non-0 even when it dispatched no events,<br>
> I have to say the thing about display queue, don't I?<br>
<br>
</span>Huh. I wonder if that should be fixed in code instead?<br>
Could anything outside libwayland-client care about when the hidden<br>
internal display queue dispatches? Errors maybe?<br></blockquote><div><br></div><div>Probably nothing external cares about the display queue, so the fix would lead<br>to 'clearer' API.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Dispatching the display queue cannot create requests one might need to<br>
flush, right?<br></blockquote><div><br></div><div>Nope atm.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Hmm, but the correction you did to the doc is good, accurate to the<br>
code AFAIK, and not doing a non-obvious change in the code is probably<br>
for the better. Yeah, nevermind, I think. :-)<br>
<br>
<br>
Thanks,<br>
pq<br></blockquote><div><br></div><div>You made me thinking if it would be better to fix it in the code... but it is probably better to not change the API<br></div><div>if it is not necessary (and it seems it is not necessary since nobody has complained about it yet)<br><br></div><div>Thanks,<br></div><div>Marek<br></div><div> </div></div><br></div></div>