Thanks John.

I was tempted to modify the autoconf script in 
order to support the pthreads library automatically,
In fact, I was evaluating the ACX_PTHREAD too (that one
>From Steven G. Johnson) but found issues. Some platforms 
require a special compiler to support pthreads. 
AIX for example. That means, in order to accommodate changes,
all autoconf scripts should be rewritten. Too risky for
our beloved stability. And since this is not a key feature 
at all, I just switched the toggle off by default. 
Let`s time pass to check if multithreading works as 
expected. If so, we could mark lcms.pc as thread safe.

Regards
Marti

-----Original Message-----
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] On Behalf Of John Cupitt
Sent: Saturday, December 30, 2006 12:53 PM
Cc: lcms-user@lists.sourceforge.net
Subject: Re: [Lcms-user] pthread support in lcms 1.16

On 12/29/06, Marti Maria <[EMAIL PROTECTED]> wrote:
> > And especially if that meant that
> > it is now safe to share cmsHTRANSFORM between threads ?
>
> Yep.

Great! Our project has been waiting on this too.

> conflicts. For the remaining 0,1% you may turn
> on the pthread support by editing lcms.h, but in
> my testing it worked fine without that.

It's not so easy for users of lcms to decide whether the installed
version is thread-safe. And I think lcms.pc needs to know about this
too: it should add -lpthreads to the library line, and possibly
-D_REENTRANT to the cflags.

How about testing for threads in configure.in (with a switch to turn
off threads if required) and if they are found, setting
LCMS_THREADSAFE (for example). I used to use the ACX_PTHREAD macro
when I had my own thread wrapper, perhaps there's something better
now. I could try to make a patch if you like.

Another solution would be to use a thread wrapper library like
gthread, though of course I can understand reluctance to make another
dependency.

John

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Lcms-user mailing list
Lcms-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lcms-user


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Lcms-user mailing list
Lcms-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lcms-user

Reply via email to