[PATCH libX11] config: Add an option to initialize threads by default.
Tiago Vignatti
tiago.vignatti at nokia.com
Mon Jan 31 05:12:29 PST 2011
On 01/31/2011 01:13 PM, ext Julien Cristau wrote:
> On Mon, Jan 31, 2011 at 12:46:56 +0200, Rami Ylimäki wrote:
>
>> This change makes it possible to guard a system against a missing
>> XInitThreads call in X clients. One might argue that this is a client
>> problem and that all X clients should call XInitThreads if it's
>> possible that they could use Xlib from multiple threads. However,
>> experience has shown that it's just too easy for developers to
>> overlook the need for this call.
>>
> This change means that somebody developing their apps on a system with
> it on will never see they need to call XInitThreads(), and stuff will
> break when moving to an Xlib without that option. I don't think that's
> a good idea.
I do agree with Julien in the sense that developers could become lazy
and in general won't care much anymore about initializing thread support.
OTOH, if you pick a Qt application for instance, you will see that it is
close to impossible to track from a stack trace of 80 functions whether
XInitThreads() is called properly or not. At the same time, it's not a
big deal to maintain the support Rami made in libX11. So why not?
Acked-by: Tiago Vignatti <tiago.vignatti at nokia.com>
Tiago
More information about the xorg-devel
mailing list