Hi

On 7/9/24 19:53, David CARLIER wrote:
Agreed, I ll likely just commit as is sometime this week.

I hereby formally claim that this is not a simple self-contained feature. It does require an RFC.

The addition of global `IMAGE_CMP_*` constants effectively introduces an entirely new "namespace" within the existing list of constants and thus I do not believe that consistency with the existing GD functionality is a valid reason. As such I believe they should follow the https://wiki.php.net/rfc/namespaces_in_bundled_extensions RFC and be placed inside the `Gd` namespace - or ideally be replaced by a proper enum or so.

Best regards
Tim Düsterhus

Reply via email to