# [Pre-Vote Announcement] Move RNG functions Random Extension

- Thread: https://externals.io/message/116124
- Original archive: https://news-web.php.net/php.internals/116124
- Started by Go Kudo on 2021-09-22T11:21:00+00:00
- Messages: 5
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #116124 — Go Kudo — 2021-09-22T11:21:00+00:00

URL: https://externals.io/message/116124#116124

The voting phase for the following RFCs will begin as soon as two weeks
have passed.

https://externals.io/message/115975

I don't see any particular discussion, so I'm contacting you just in case.
Best regards.

Go Kudo

---

## #116125 — Aleksander Machniak — 2021-09-22T11:51:32+00:00

URL: https://externals.io/message/116124#116125 · In reply to: #116124

> The voting phase for the following RFCs will begin as soon as two weeks
> have passed.

The RFC title is not correct English (missing "to" or "into"). The first
paragraph in Introduction is also hard to read.

-- 
Aleksander Machniak
Kolab Groupware Developer        [https://kolab.org]
Roundcube Webmail Developer  [https://roundcube.net]
----------------------------------------------------
PGP: 19359DC1 # Blog: https://kolabian.wordpress.com

---

## #116212 — Nikita Popov — 2021-10-04T10:21:06+00:00

URL: https://externals.io/message/116124#116212 · In reply to: #116124

> The voting phase for the following RFCs will begin as soon as two weeks
> have passed.
>
> https://externals.io/message/115975
>
> I don't see any particular discussion, so I'm contacting you just in case.
>

I'm pretty neutral on this proposal. I don't think there's a strong
motivation to move this functionality into a separate extension, but I also
don't see any particular problems with doing it (as long as it's an
always-required extension, which this is).

I believe Levi is a proponent of splitting up ext/standard into smaller
extensions, maybe he has some thoughts on this.

Regards,
Nikita

---

## #116235 — Levi Morrison via internals — 2021-10-06T12:52:33+00:00

URL: https://externals.io/message/116124#116235 · In reply to: #116212

>
> 
> > […]
>
>
> I'm pretty neutral on this proposal. I don't think there's a strong motivation to move this functionality into a separate extension, but I also don't see any particular problems with doing it (as long as it's an always-required extension, which this is).
>
> I believe Levi is a proponent of splitting up ext/standard into smaller extensions, maybe he has some thoughts on this.
>
> Regards,
> Nikita

I've glanced at the patch; it seems fine. I do like smaller, focused
extensions better than throwing things in `ext/standard`.

Can we change the return types of the functions in this extension
returning `SUCCESS`/`FAILURE` to `zend_result` at the same time? I
think it's just these two:

  - php_random_bytes
  - php_random_int

Actually, is that something we could still do for PHP 8.1?

---

## #116241 — Go Kudo — 2021-10-07T13:05:55+00:00

URL: https://externals.io/message/116124#116241 · In reply to: #116125

2021年9月22日(水) 21:01 Aleksander Machniak <alec@alec.pl>:

> 
> > […]
>
> The RFC title is not correct English (missing "to" or "into"). The first
> paragraph in Introduction is also hard to read.
>
> --
> Aleksander Machniak
> Kolab Groupware Developer        [https://kolab.org]
> Roundcube Webmail Developer  [https://roundcube.net]
> ----------------------------------------------------
> PGP: 19359DC1 # Blog: https://kolabian.wordpress.com
>
> --
> 
> To unsubscribe, visit: https://www.php.net/unsub.php
>
>
Sorry.

I'm not native English speaker and I am not good at English.
I have improved the title first.

I don't know how to fix the first chapter, so I left it as it is.

If this causes problems in the voting phase of the RFC, I will try to fix
it further, how about that?