mbstring: Do functions marked “TODO Return void” require an RFC to change return type?

Hi,

In ext/mbstring/php_mbregex.c, I noticed several places with comments like:

// TODO Return void
RETURN_TRUE;

ext/mbstring/php_mbregex.c:1567

This suggests the intent is to change these functions to return void instead of bool.

Before preparing a patch, I wanted to confirm the process:

Would changing these functions from returning true to void (i.e., RETURN_NULL() in C and updating stubs to: void) require an RFC?
Or is this considered a minor/internal API cleanup that can go directly as a PR?

I’m aware that this could have a userland impact if the return value is currently used, so I wanted to check what the preferred approach is.

Happy to prepare a PR once the direction is clarified.

Thanks!

Bắt đầu tại ext/mbstring/php_mbregex.c:1567 và xem xét các chú thích TODO xung quanh cùng các function stub liên quan được đề cập trong issue. Xác định xem việc thay đổi kiểu trả về có ảnh hưởng đến userland API và cần một RFC hay không; issue được hoàn tất khi quy trình và phạm vi của dự án đã được xác nhận trước khi chuẩn bị một patch.