X Tutup
Skip to content

IMAGE: Use palette class in more codecs#6469

Merged
bluegr merged 1 commit intoscummvm:masterfrom
OMGPizzaGuy:codec-palette
Mar 8, 2025
Merged

IMAGE: Use palette class in more codecs#6469
bluegr merged 1 commit intoscummvm:masterfrom
OMGPizzaGuy:codec-palette

Conversation

@OMGPizzaGuy
Copy link
Member

@OMGPizzaGuy OMGPizzaGuy commented Mar 6, 2025

These changes are simple, but not all of it is tested. I am having difficulty finding examples demos with videos using these codecs.

Currently tested:
CDToons - Testbed video with Where in the USA is Carmen Sandiego? (Windows Demo)

@OMGPizzaGuy OMGPizzaGuy requested a review from ccawley2011 March 6, 2025 02:45
Copy link
Member

@bluegr bluegr left a comment

Choose a reason for hiding this comment

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

LGTM

@OMGPizzaGuy OMGPizzaGuy removed the request for review from ccawley2011 March 6, 2025 23:59
@OMGPizzaGuy
Copy link
Member Author

OMGPizzaGuy commented Mar 7, 2025

Remaining codecs were tested with testbed forced to CLUT8 pixel format. Test videos were from samples.mplayerhq.hu

@bluegr
Copy link
Member

bluegr commented Mar 8, 2025

Nice work, thanks!

@bluegr bluegr merged commit 35b005b into scummvm:master Mar 8, 2025
8 checks passed
@OMGPizzaGuy OMGPizzaGuy deleted the codec-palette branch July 1, 2025 01:45
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.

3 participants

X Tutup