[PHP-DEV] Re: [PHP-CVS] cvs: php4 /ext/sysvshm sysvshm.c

[derick]

On Thu, 22 Nov 2001, Sterling Hughes wrote:

> > while fixing a bug in this extension, I noticed that this extension does
> > not use any of the advanced Zend API functions (ie. resource management).
>
>   it does do resource management.

But not with ZEND_FETCH and ZEND_REGISTER_RESOURCE, which include some
additional error checking.

>   So change it to use the new Zend API. :)

If you can make my days 48 hours :)

Derick


-- 
PHP Development Mailing List 
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]

[PHP-DEV] Re: [PHP-CVS] cvs: php4 /ext/sysvshm sysvshm.c

[Sterling Hughes]

On Thursday 22 November 2001 16:18, [EMAIL PROTECTED] wrote:
> Hello,
>
> while fixing a bug in this extension, I noticed that this extension does
> not use any of the advanced Zend API functions (ie. resource management).

it does do resource management.

> This was some advanced error checking is not done. I think it would be
> wise that this is changed, so that the extension is more maintainable in
> the future.
>

So change it to use the new Zend API. :)

-Sterling

-- 
PHP Development Mailing List 
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]

[PHP-DEV] Re: [PHP-CVS] cvs: php4 /ext/sysvshm sysvshm.c

[derick]

Hello,

while fixing a bug in this extension, I noticed that this extension does
not use any of the advanced Zend API functions (ie. resource management).
This was some advanced error checking is not done. I think it would be
wise that this is changed, so that the extension is more maintainable in
the future.

Derick


-- 
PHP Development Mailing List 
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]