Skip to content

[ME][2.5/7] Simplify transaction adapters - #408

Merged
starstormhun merged 1 commit into
IllusionMods:masterfrom
tomTom1010-IEE:me-upstream-025-transactions
Sep 13, 2026
Merged

starstormhun merged 1 commit into
IllusionMods:masterfrom
tomTom1010-IEE:me-upstream-025-transactions

Conversation

@tomTom1010EEE

Copy link
Copy Markdown
Contributor

Description

  • Remove the Cubemap storage and record-access delegate containers; use existing Core storage types directly.
  • Replace Texture2D field-access delegates with direct assignments, keeping animation-binding rollback shared.
  • Keep Studio property and texture loading together in one file.

Motivation and Context

Small follow-up to #407 to reduce indirection. One commit based on the merged upstream version; no later maintenance stages or UI changes. Save formats and public APIs are unchanged.

How Has This Been Tested?

All seven targets built successfully. Metadata tests and managed transaction checks passed. The rebased source tree matches the tested tree. No in-game validation of this follow-up yet.

Screenshots (if appropriate):

No visual changes.

Types of changes

  • Bug fix
  • New feature
  • Breaking change
  • Refactor / maintenance (no intended behavior change)

@starstormhun starstormhun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A bit of moving around code, and some simplifying indeed.

@starstormhun
starstormhun merged commit c65edcb into IllusionMods:master Sep 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants