Re: Patch for ldap.c with binary values support
| From: | Jani Taskinen | Date: | Tue, 29 Aug 2000 11:10:59 +0000 |
| Subject: | Re: Patch for ldap.c with binary values support | ||
| References: | 1 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-31048@lists.php.net to get a copy of this message | ||
On Tue, 29 Aug 2000, Stig Venaas wrote:
>On Tue, Aug 29, 2000 at 04:36:59PM +1100, terrence_miao@email.com wrote:
>>
>> Find the patch file for ldap.c (PHP 4.0.1pl2) with LDAP
>> binary values support:
>>
>> ftp://igloo.its.unimelb.edu.au/pub/Linux/PHP4/
>
>I'm not sure what to think of this.
>Good points:
>
>Ease of use, one can easily handle both text and binary data at the
>same time, to use ldap_get_values_len for all binary data might be
>awkward.
>Bad points:
>
>There is some extra overhead and in most cases people will have only
>text and want maximum performance.
Please check latest CVS. This patch is already been applied..
>the performance hit is close to 0 (might be) then that may not be an
>issue.
I haven't noticed any performance loss with this patch.
--Jani