Skip to content
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

avm2: Work towards starting to remove PrimitiveObject #18198

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

Lord-McSweeney
Copy link
Collaborator

@Lord-McSweeney Lord-McSweeney commented Oct 6, 2024

Fixes #17859, should fix #15559, fixes #11432

This PR

  1. Introduces methods on Value that allow for doing most only Class-related operations with it without boxing it (commits 1-2)
  2. Completely removes coerce_to_object outside of core code (commits 3-10)

@danielhjacobs danielhjacobs added A-avm2 Area: AVM2 (ActionScript 3) T-fix Type: Bug fix (in something that's supposed to work already) labels Oct 7, 2024
@Lord-McSweeney Lord-McSweeney force-pushed the avm2-primobj-1 branch 3 times, most recently from f6fd011 to f3cc4a6 Compare October 10, 2024 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-avm2 Area: AVM2 (ActionScript 3) T-fix Type: Bug fix (in something that's supposed to work already)
Projects
None yet
3 participants