Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:97192
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.223.195 as permitted sender)
MIME-Version: 1.0
Sender: kalle.php@gmail.com
In-Reply-To: <5d865bc1-dc06-54b8-800e-99f1c2914632@cubiclesoft.com>
References: <56e89e82-00aa-5521-f014-14cf4d588157@cubiclesoft.com>
 <CAJW__o2vn0PGJEYQ8-qojYCBTd7Czjf193NEzzaCqK+NHEHZ6A@mail.gmail.com> <5d865bc1-dc06-54b8-800e-99f1c2914632@cubiclesoft.com>
Date: Fri, 25 Nov 2016 21:43:02 +0100
Message-ID: <CAJW__o3qa=Fio_ZenLvVhL=DXiD=e-r8haxUcLEq2pjVib8xwQ@mail.gmail.com>
To: Thomas Hruska <thruska@cubiclesoft.com>
Cc: PHP Development <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] Translating optional zval ** with zend_parse_parameters()
 to PHP 7
From: kalle@php.net (Kalle Sommer Nielsen)

2016-11-25 13:30 GMT+01:00 Thomas Hruska <thruska@cubiclesoft.com>:
> I need to be able to *modify* the original variable that was passed in.  I already use "|l" and zend_long
> elsewhere in the extension

Ah my bad I misread it!

In that case take a look at where such is implemented in php-src,
dns_get_record() is an example despite the slightly clouded code:
http://git.php.net/?p=php-src.git;a=blob;f=ext/standard/dns.c;h=f92015eee90d3e93a801e93d6381d89923825166;hb=refs/heads/master#l1011

See $weight_list:

- Check if the argument is passed
- zval_dtor()
- Populate it (ZVAL_LONG is the same)

The main difference from your code is to use 'z' (lower case) in ZPP,
and then use zval* (which is what 'z' returns).

I'm not at my dev machine, so I cannot test it but that should be how it goes

-- 
regards,

Kalle Sommer Nielsen
kalle@php.net