Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120874
MIME-Version: 1.0
References: <CABLJ+15-DMEsUpBdhHLK8zThA==zNU=WqM0Tz-aK+V1p2N3Ynw@mail.gmail.com>
 <CAH5C8xVJZ1xtJL8rLs5ZsEJ-FmkxBjf5qLw7bgeQ+d++uo3kbw@mail.gmail.com>
In-Reply-To: <CAH5C8xVJZ1xtJL8rLs5ZsEJ-FmkxBjf5qLw7bgeQ+d++uo3kbw@mail.gmail.com>
Date: Mon, 31 Jul 2023 13:12:46 -0700
Message-ID: <CABLJ+16wUYrEyUK61XajKoxJiFdtgTsrs=VTn7WTXH2HK0Ko=A@mail.gmail.com>
To: =?UTF-8?B?TcOhdMOpIEtvY3Npcw==?= <kocsismate90@gmail.com>
Cc: PHP Internals List <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] Arginfo Mismatch For Optional Parameters
From: sutabi@gmail.com (Joseph Montanez)

M=C3=A1t=C3=A9,

Thanks! That was the issue :). All of the c code, including arginfo
and stubs are auto generated on my end. I didn't know you could use
stubs to generate the arg info, so will need to look into it later.
Just for reference the following works:

ZEND_BEGIN_ARG_INFO_EX(arginfo_vector3__construct, 0, 0, 0)
    ZEND_ARG_TYPE_MASK(0, x, MAY_BE_DOUBLE|MAY_BE_NULL, "0")
    ZEND_ARG_TYPE_MASK(0, y, MAY_BE_DOUBLE|MAY_BE_NULL, "0")
    ZEND_ARG_TYPE_MASK(0, z, MAY_BE_DOUBLE|MAY_BE_NULL, "0")
ZEND_END_ARG_INFO()


Thanks,
Joseph Montanez

On Mon, Jul 31, 2023 at 12:38=E2=80=AFPM M=C3=A1t=C3=A9 Kocsis <kocsismate9=
0@gmail.com> wrote:
>
> Hi,
>
> The problem is that your arginfo declares parameters of type double, whil=
e you accept the double|null type in ZPP.
>
> P.s. i'm not sure based on your message whether you use stubs for declari=
ng symbols, but if that's not the case, then doing so is highly recommended=
. You can find plenty of examples in php-src (*.stub.php files).
>
> Regards,
> M=C3=A1t=C3=A9