-
Notifications
You must be signed in to change notification settings - Fork 5
Mupen-owned D2D brushes #586
Copy link
Copy link
Open
Labels
bugThis issue describes unexpected behaviour.This issue describes unexpected behaviour.
Milestone
Description
Rationale
Having Lua-owned D2D brushes was a terrible idea. They're easy to leak, caching them involves Lua-side tables and lookups... Terrible.
Proposal Summary
Mupen should own D2D brushes and lend lightuserdata to Lua.
Open Questions
No response
Technical Considerations
This change sounds like it's breaking, but it doesn't have to be. We can just stub out d2d.delete_brush.
Is it worth doing it like that, or should we break the D2D API?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugThis issue describes unexpected behaviour.This issue describes unexpected behaviour.