Re: GTK frame changes

[grischka]

Jan Djärv wrote:
> grischka skrev:
> > Jan Djärv wrote:
> > > grischka skrev:
> > > > Jan Djärv <address@hidden> wrote in Changelog:
> > > >
> > > > > xg_frame_set_char_size): Do not set pixel width/height here or
> > > > > call change_frame_size. Just call flush_and_sync.
> > > > > (flush_and_sync): Reintroduced.
> > > >
> > > > Nice work and the idea is right IMO, just the problem remains that
> > > > "flush_and_sync()" does not guarantee that the resize request was
> > > > actually handled,  i.e. that a ConfigureNotify was received before
> > > > flush_and_sync() returns.  In my tests in ~70% of runs this is not
> > > > the case.
> > >
> > > No, it is not certain.  But if the X server handles requests from 
> > > clients in sequence (which it should), it is certain.  
> >
> > What makes you think that flush_and_sync would work if requests are
> > handled sequentially by the X server?
>
> XSync sends a request to the server and waits for a reply.  The resize
> request is sent before that, so the resize should be handeled before the 
> XSync reply is sent.

Yes, but this reply is something in the sense of the X protocol, not
in the sense of ConfigureNotify.  It just means that the X server got
the resize message when XSync returns.   It doesn't mean at all that
a ConfigureNotify is already in the client queue let alone that emacs
now knows how big it is.

--- grischka

>     Jan D.