Skip to content

avm1: Migrate flash.geom.Matrix to pure ActionScript - #24831

Merged
kjarosh merged 4 commits into
masterfrom
avm1_matrix
Oct 6, 2026
Merged

kjarosh merged 4 commits into
masterfrom
avm1_matrix

Conversation

@relrelb

@relrelb relrelb commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Description

Similar to #24736 and #24743, but for flash.geom.Matrix.

Testing

Converted the avm1/matrix test to Rascal and expanded it.

Also extracted a shared test_utils.as file to de-duplicate common helper functions between different tests.

Checklist

  • I, a human, have self-reviewed this PR and fully understand the changes within.
  • I have made or updated tests where possible.
  • All of my commits are properly scoped, compile successfully, and pass all tests.
  • This PR does not make sense to split up into smaller PRs.
  • An LLM was involved in the authoring of this code.

@kjarosh

kjarosh commented Oct 3, 2026

Copy link
Copy Markdown
Member

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

An LLM model / agent cannot legally hold authorship and we cannot contact it. If for some reason you want to mention what model you were using to produce the commit, please don't use the Co-Authored-By tag. See also #24521. Note that Ruffle does not allow fully autonomous code contributions and requires a human in the loop (see CONTRIBUTING.md), in which case that human should be listed as the author.

Comment thread tests/tests/swfs/avm1/_shared/test_utils.as Outdated
@kjarosh kjarosh added A-avm1 Area: AVM1 (ActionScript 1 & 2) T-compat Type: Compatibility with Flash Player labels Oct 3, 2026
@relrelb
relrelb requested a review from kjarosh October 6, 2026 15:51
Give the generic helpers in avm1/rectangle the same definitions as in
avm1/matrix and avm1/point, so they can later be de-duplicated. The
tracingMethod helper now prints call arguments and tracingNumber takes
a label, so the expected output is re-recorded from Flash Player.
@kjarosh
kjarosh enabled auto-merge (rebase) October 6, 2026 20:54
@kjarosh
kjarosh merged commit a407c51 into master Oct 6, 2026
27 checks passed
@kjarosh
kjarosh deleted the avm1_matrix branch October 6, 2026 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-avm1 Area: AVM1 (ActionScript 1 & 2) T-compat Type: Compatibility with Flash Player

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants