Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:111879
MIME-Version: 1.0
References: <20200917145304.4a6cfe45@mcmic-probook.opensides.be>
In-Reply-To: <20200917145304.4a6cfe45@mcmic-probook.opensides.be>
Date: Thu, 17 Sep 2020 15:03:05 +0200
Message-ID: <CAFPFaMKCoLuYfZfdXqG=tp5EkLoODuOjNPEB2HaLv_p54Ar=2Q@mail.gmail.com>
To: =?UTF-8?Q?C=C3=B4me_Chilliet?= <come.chilliet@fusiondirectory.org>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000f6e63605af82013f"
Subject: Re: [PHP-DEV] Stubs in ext/ldap
From: george.banyard@gmail.com ("G. P. B.")

--000000000000f6e63605af82013f
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, 17 Sep 2020 at 14:53, C=C3=B4me Chilliet <
come.chilliet@fusiondirectory.org> wrote:

> Hello,
>
> After playing with phpstan and seeing their stubs for some LDAP functions
> were
>  wrong, I noticed that documentation for LDAP functions and stubs in
>  ext/ldap/ldap.stub.php differs.
>

A lot of the documentation is not up to date and will needs to be updated
after
there has been a check of the argument names for consistency with other
extensions in regards to named params, the stubs are the source of trust.


> Especially in a lot of functions, the documentation states "string $dn =
=3D
> NULL"
>  but the stub states non-nullable string parameter. Also, the ldap.stub.p=
hp
>  file contains a lot of "string $bind_rdn =3D UNKNOWN" in places where th=
e
>  documentations states =3D NULL. What does UNKOWN mean here?
>

UNKNOWN means that the default value cannot be specified due to some reason
mostly the case for arguments passed by references or funcky function
signature
implementation which check the number of argument passed to the function.


> I would like to fix those conflicts.
> Basically my questions are:
> 1) What is the difference between s and S in zend_parse_parameters and ho=
w
> do
>  they behave for a NULL value?
>

None in regards to userland as 's' is a char* followed by a size_t ad 'S'
is a zend_string.
As for all internal functions which are not nullable they will coerce null
to the respective type
so for a string argument null would become an empty string.


> 2) What would changing =3D UNKNOWN to =3D NULL impact?

Explicit support for nullable types in the C implementation


> 3) When a function like ldap_bind is called, is calling ldap_bind($link),
> and
>  ldap_bind($link, NULL) changing anything else than ZEND_NUM_ARGS?
>

This relates to point 1 and 2 above, if there is no support for nullable
types
passing null can change the meaning of it.

C=C3=B4me
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: https://www.php.net/unsub.php
>
>
Hope this answers most of the questions

Best regards

George P. Banyard

--000000000000f6e63605af82013f--