Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-20 Thread Pierre Joye
On Wed, Aug 20, 2014 at 7:24 AM, Dmitry Stogov dmi...@zend.com wrote: Anyone objects about it or thinks it needs RFC? No, goahead :) -- Pierre @pierrejoye | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-20 Thread Julien Pauli
On Wed, Aug 20, 2014 at 7:24 AM, Dmitry Stogov dmi...@zend.com wrote: Anyone objects about it or thinks it needs RFC? Thanks. Dmitry. On Wed, Aug 20, 2014 at 6:48 AM, Laruence larue...@php.net wrote: Hey: On Tue, Aug 19, 2014 at 10:10 PM, Dmitry Stogov dmi...@zend.com wrote: Hi,

Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-20 Thread Dmitry Stogov
The are not defined. They are open (or closed but not freed yet). You probably meant get_defined_resorce_types() Thanks. Dmitry. On Wed, Aug 20, 2014 at 1:12 PM, Julien Pauli jpa...@php.net wrote: On Wed, Aug 20, 2014 at 7:24 AM, Dmitry Stogov dmi...@zend.com wrote: Anyone objects about it

Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-20 Thread Markus Fischer
Hello Dmitry, On 20.08.2014 12:08, Dmitry Stogov wrote: To have a robust way to detect resource leaks I propose a new function - get_resources() that returns an array of all registered resources or an array of registered resources of particular type. See patch:

Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-20 Thread Stas Malyshev
Hi! array of registered resources of particular type. See patch: https://gist.github.com/dstogov/f96c04f5979e726909ab It would be better as a pull, it's be easier to comment on it. For the function get_resources, the proto says it has no arguments, but it actually has one S argument that is

Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-20 Thread Dmitry Stogov
done. Dmitry. On Thu, Aug 21, 2014 at 12:50 AM, Stas Malyshev smalys...@sugarcrm.com wrote: Hi! array of registered resources of particular type. See patch: https://gist.github.com/dstogov/f96c04f5979e726909ab It would be better as a pull, it's be easier to comment on it. For the

[PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-19 Thread Dmitry Stogov
Hi, Yesterday we discussed with Nikta the failure of ext/standard/tests/http/bug60570.phpt. It was in context of AST patch, but the failure is not related to AST at all. It's just a bad test that makes wrong assumption. Resource leaks can't be caught using get_memory_usage(). To have a robust

Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-19 Thread Pierre Joye
hi, On Tue, Aug 19, 2014 at 4:10 PM, Dmitry Stogov dmi...@zend.com wrote: Hi, Yesterday we discussed with Nikta the failure of ext/standard/tests/http/bug60570.phpt. It was in context of AST patch, but the failure is not related to AST at all. It's just a bad test that makes wrong

Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-19 Thread Laruence
Hey: On Tue, Aug 19, 2014 at 10:10 PM, Dmitry Stogov dmi...@zend.com wrote: Hi, Yesterday we discussed with Nikta the failure of ext/standard/tests/http/bug60570.phpt. It was in context of AST patch, but the failure is not related to AST at all. It's just a bad test that makes wrong

Re: [PHP-DEV] ext/standard/tests/http/bug60570.phpt failure

2014-08-19 Thread Dmitry Stogov
Anyone objects about it or thinks it needs RFC? Thanks. Dmitry. On Wed, Aug 20, 2014 at 6:48 AM, Laruence larue...@php.net wrote: Hey: On Tue, Aug 19, 2014 at 10:10 PM, Dmitry Stogov dmi...@zend.com wrote: Hi, Yesterday we discussed with Nikta the failure of