Re: Fw: Bug of the day (set_concurrency)

["Kurt D. Zeilenga" <Kurt@OpenLDAP.org>]

As a matter of policy, we do not accept third party patches.  Patches
must be submitted by the original author per our contributing guidelines.
  http://www.openldap.org/devel/contributing.html

When someone send you a patch, politely ask them to submit the
patch to the project per these guidelines.  Thanks.

Kurt

At 04:21 PM 7/6/00 +0100, Al Sutton wrote:
>Heres a patch based on Marks solution (outlined below).
>
>Al.
>
>----
>Al Sutton (al@alsutton.com)
>WWW: http://www.buddysites.net/
>ICQ: 78617500
>
>----- Original Message -----
>From: "Mark Adamson" <adamson@andrew.cmu.edu>
>To: "Al Sutton" <al@alsutton.com>
>Sent: Thursday, July 06, 2000 2:12 PM
>Subject: Re: Bug of the day (set_concurrency)
>
>
>> Yeah, I saw that one too. The things that get #define'd in the portable.h
>> file do not mesh entirely with what is in the code. Chances are that like
>> on my machine there is no setconcurrency or getconcurrency calls in the
>> pthread libraries. And using this concurrency stuff is just an option from
>> the config file anyway, one that I am not using.
>>
>> If you read the code around the ldap_pvt_thread_set_concurrency()
>> functions in libraries/libldap_r/ you'll see it's not real clear what is
>> supposed to be going on. There are ifdef's within ifdef's that depend on
>> the same macro, and areas of code that can never happen due to the
>ifdef's.
>>
>> I went into servers/slapd/config.o and put this around the set_concurrency
>> call:
>>
>>   #ifdef HAVE_PTHREAD_SETCONCURRENCY
>>
>>   #endif
>>
>>
>>
>> -Mark Adamson
>>  Carnegie Mellon
>>
>>