bool ldap_passwd(resource $link, string $user, string $newpw [, string $oldpw]);

• Move $oldpw after $newpw, because it is optional.
• Remove $genpw. Maybe create another function for that usecase eg. bool ldap_passwd_generate(resource $link, string $user, string &$newpw [, string $oldpw]);

string ldap_whoami(resource $link);

• Return username without a reference.

Why ?

$newpw is optional as well.

And for ldap_exop_whoami, same question, what is wrong with bool ldap_exop_whoami($link, &$identity) ?
It is consistent with https://secure.php.net/manual/en/function.ldap-get-option.php

Note that the RFC was announced on the ML, feel free to comment on the ML with some explanations.

Please don't forget to squash changes before merging.

I'm not sure why this was merged, the RFC is a draft ?

@nikic Can you point to the documentation about that? I tried to follow the workflow described on https://wiki.php.net/vcs/gitworkflow#merge_a_pull_request
@krakjoe Because one of the only answers on the internal mailing list I got was «I don't think it is necessary to have RFC for such specific functions» and no-one objected.

Also feature freeze is on July 20 and I wanted to merge before that.