Skip to content

ext/gd: Fix incorrect argument numbers in GD affine function errors - #23356

Merged
LamentXU123 merged 3 commits into
php:PHP-8.4from
LamentXU123:argnum
Aug 19, 2026
Merged

ext/gd: Fix incorrect argument numbers in GD affine function errors#23356
LamentXU123 merged 3 commits into
php:PHP-8.4from
LamentXU123:argnum

Conversation

@LamentXU123

Copy link
Copy Markdown
Member

This fixes wrong argument in error message in imageaffinematrixget and mageaffinematrixconcat

@LamentXU123
LamentXU123 requested a review from devnexen as a code owner August 18, 2026 18:38
@LamentXU123
LamentXU123 marked this pull request as draft August 18, 2026 18:38
@LamentXU123
LamentXU123 changed the base branch from master to PHP-8.4 August 18, 2026 18:39
@LamentXU123
LamentXU123 marked this pull request as ready for review August 18, 2026 18:39
@Girgias

Girgias commented Aug 18, 2026

Copy link
Copy Markdown
Member

It would probably make sense for master to create a custom ZPP specifier for array|float as currently it doesn't behave like that type union (weak and strict mode behaviour is a bit nonsensical here)

Comment thread ext/gd/tests/imageaffinematrixconcat_error.phpt Outdated
Co-authored-by: NickSdot <32384907+NickSdot@users.noreply.github.com>
@LamentXU123

LamentXU123 commented Aug 18, 2026

Copy link
Copy Markdown
Member Author

It would probably make sense for master to create a custom ZPP specifier for array|float as currently it doesn't behave like that type union (weak and strict mode behaviour is a bit nonsensical here)

This makes sense in the long-term. Perhaps another good first issue here.
Okay I will take this as a chance to learn more about the engine.

@LamentXU123
LamentXU123 merged commit aad2b53 into php:PHP-8.4 Aug 19, 2026
18 checks passed
@LamentXU123
LamentXU123 deleted the argnum branch August 19, 2026 05:40
LamentXU123 added a commit that referenced this pull request Aug 19, 2026
* PHP-8.5:
  ext/gd: Fix incorrect argument numbers in GD affine function errors (#23356)
pull Bot pushed a commit to ppker/php-src that referenced this pull request Aug 19, 2026
* PHP-8.4:
  ext/gd: Fix incorrect argument numbers in GD affine function errors (php#23356)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants