KEMBAR78
DIRECTOR: More fixes for D5 by moralrecordings · Pull Request #4955 · scummvm/scummvm · GitHub
Skip to content

Conversation

@moralrecordings
Copy link
Contributor

No description provided.

@moralrecordings moralrecordings changed the title DIRECTOR: More fixes for V5 DIRECTOR: More fixes for D5 Apr 30, 2023
@moralrecordings moralrecordings force-pushed the director_v5 branch 2 times, most recently from cc25b37 to 7fe471a Compare May 4, 2023 07:42
In D5 and up, you can have multiple casts in the same movie, along with
external casts. Unlike shared casts these are different namespaces;
each movie maps the casts to a libId, which is then used by
CastMemberID.castLib.

For D4 and below, the default cast libId is now represented by
DEFAULT_CAST_LIB. This should make refactoring easier in future.
Previously, the palette ID namespace was shared between cast ID, builtin
ID and resource fork ID. This was quite messy.

The new approach is to use CastMemberID; builtin palettes are libId -1,
cast palettes use their respective libId, and a null CastMemberID
indicates a blank entry.
@sev-
Copy link
Member

sev- commented May 10, 2023

Awesome!

@sev- sev- merged commit 8976c4f into scummvm:master May 10, 2023
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