Skip to content

avm1: migrate StageObject to NativeObject #20027

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Apr 12, 2025

Conversation

moulins
Copy link
Contributor

@moulins moulins commented Apr 6, 2025

The display objects (Avm1Button, EditText, MovieClip, Video) are directly stored as NativeObject variants of their ScriptObjects, and the StageObject-specific data (namely, the AVM1 text field binding list) is moved directly inside the DO structs.

After this, only SuperObject will be left as a Object variant.

@moulins moulins added A-avm1 Area: AVM1 (ActionScript 1 & 2) T-refactor Type: Refactor / Cleanup labels Apr 6, 2025
@moulins moulins force-pushed the avm1-native-stage-object branch 3 times, most recently from 3a5b7a5 to de66cf7 Compare April 9, 2025 00:59
@Lord-McSweeney
Copy link
Collaborator

Shouldn't the text field binding functions be passed a DisplayObject instead of an &impl TDisplayObject?

@moulins
Copy link
Contributor Author

moulins commented Apr 10, 2025

Shouldn't the text field binding functions be passed a DisplayObject instead of an &impl TDisplayObject?

Good point, I don't remember exactly why I did this, but it clearly serves no purpose in the final version of the code ^^

EDIT: now fixed.

@moulins moulins force-pushed the avm1-native-stage-object branch 2 times, most recently from 7e3277c to cc20bc3 Compare April 10, 2025 21:12
@moulins moulins force-pushed the avm1-native-stage-object branch from cc20bc3 to edfa66d Compare April 12, 2025 01:09
@moulins moulins merged commit d9b8812 into ruffle-rs:master Apr 12, 2025
22 checks passed
@moulins moulins deleted the avm1-native-stage-object branch April 12, 2025 01:47
moulins added a commit to moulins/ruffle that referenced this pull request Apr 13, 2025
moulins added a commit to moulins/ruffle that referenced this pull request Apr 13, 2025
moulins added a commit to moulins/ruffle that referenced this pull request Apr 13, 2025
moulins added a commit that referenced this pull request Apr 13, 2025
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-refactor Type: Refactor / Cleanup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants