Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:81579
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.215.45 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CABBJUpdzJZFYwBh0trYMTFSBCDrjH66nJFrBRNhCQTGrpTRDgQ@mail.gmail.com>
References: <CABBJUpdzJZFYwBh0trYMTFSBCDrjH66nJFrBRNhCQTGrpTRDgQ@mail.gmail.com>
Date: Mon, 2 Feb 2015 18:48:31 +0800
Message-ID: <CABBJUpcQ1hN79s2dhxE+JhZY7uWh9DJFTWzk7mmtwUgA+om7dg@mail.gmail.com>
To: PHP Internals <internals@lists.php.net>, Dmitry Stogov <dmitry@zend.com>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [DICUSS]Cleanup resource handling APIs
From: laruence@php.net (Xinchen Hui)

Hey:

   if no objections, I will commit it.. (no bc breaks, no handle
removing , only APIs change)

   dmitry suggest we change zend_fetch_resource to
zend_verify_resource, I think it make sense.. will do it.


thanks
On Mon, Feb 2, 2015 at 1:34 PM, Xinchen Hui <laruence@php.net> wrote:
> Hey:
>
>      we used to use lval of zval as a handle to access resource type..
>
>      but now, we introduced a new type IS_RESOURCE, which make the
> handle(id) sort of redundant .
>
>      further more, the common usage when handling resource is like:
>
>       if (zend_parse_parameters(ZEND_NUM_ARGS(), "rl", &result,
> &offset) == FAILURE) {
>         return;
>     }
>     ZEND_FETCH_RESOURCE(mysql_result, MYSQL_RES *, result, -1, "MySQL
> result", le_result);
>
>     as you can see, we use "r" to receive a IS_RESOURCE type, that
> means, check the type in ZEND_FETCH_RESOURCE is overhead..
>
>    and:
>
>    ZEND_API void *zend_fetch_resource(zval *passed_id, int default_id,
> const char *resource_type_name, int *found_resource_type, int
> num_resource_types, ...)
>
>    we use va_args to passing resource type, that means, the rescue
> type arguments can not be passed by register but by stack.. which is a
> little low effiicient .
>
>    so, I'd like propose a zend_resource handling API cleanup..
>
>    1.  DROP ZEND_REGISTER_RESOURCE/FETCH_RESOURCE.
>
>    2.  add :
>
>        ZEND_API void *zend_fetch_resource(zend_resource *res, const
> char *resource_type_name, int resource_type);
> ZEND_API void *zend_fetch_resource2(zend_resource *res, const char
> *resource_type_name, int *found_type, int resource_type, int
> resource_type2);
> ZEND_API void *zend_fetch_resource_ex(zval *res, const char
> *resource_type_name, int resource_type);
> ZEND_API void *zend_fetch_resource2_ex(zval *res, const char
> *resource_type_name, int *found_type, int resource_type, int
> resource_type2);
>
>    a underworking patch could be found at:
> https://github.com/php/php-src/pull/1042
>
>    any ideas & objections?
>
> thanks
>
> --
> Xinchen Hui
> @Laruence
> http://www.laruence.com/



-- 
Xinchen Hui
@Laruence
http://www.laruence.com/