Hi Faraz,

On Mon, 2008-01-07 at 20:53 +0500, Faraz Khan wrote:
> Dear all,
> I'm trying to apply the php ldap EXOP patch from :
> 
> <http://www.sys-net.it/~ando/Download/#PHP>
> 
> The patch defines a zstr in ldap.c at line 1731 as obvious below. ZSTR  
> no longer appears to be defined anywhere in php or zend.h - can  
> somebody help me with this? Should i just define it manually?

I don't know that patch but a little help: zstr is a PHP 6 type for
handling unicode and binary strings. So that patch is either missing
some compatibility code or is only for PHP 6. If you want to adopt it
either define a zstr using something like

typedef union _zstr {
        char *s;
        /* void *u;   -- If you get an error due a missing u component 
                         of that union either try using a void for it
                         or change it to take .s as char* .... */
        void *v;
} zstr;

or try to change the "zstr foo"s there to "char *foo" and drop
the .s/.u/.v where that var is being used.

The original declaration of that type can be found here:
http://lxr.php.net/source/ZendEngine2/zend.h#235

Both ways aren't really good, for more I'd have to check the patch,
maybe it's best to ask the author of the patch.

johannes

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to