On Sat, May 10, 2014 at 02:55:11PM +0200, Uli Schlachter wrote: > On 10.05.2014 00:21, [email protected] wrote: > > From: "Owen W. Taylor" <[email protected]> > > > > When XCB owns the X socket, dpy->request is not updated, so > > NextRequest() and XNextRequest() return the wrong value. There's > > nothing we can do to fix NextRequest() while retaining ABI compat, > > but change XNextRequest() to grab the socket back from XCB, > > updating dpy->request. > > > > Signed-off-by: Owen W. Taylor <[email protected]> > > Reviewed-by: Uli Schlachter <[email protected]> > > (And this also makes this function thread safe on 32bit archs were reading a > 64bit value isn't atomic)
I don't have any issues with the patch, aside from possibly making people think that using NextRequest is a good idea. :-) But this comment isn't true; neither NextRequest nor XNextRequest can ever be made thread-safe in any implementation, as you could always race with another thread that's issuing a request, which invalidates NextRequest's post-conditions. Jamey
signature.asc
Description: Digital signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
