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

BGRA Fixes #44

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

BGRA Fixes #44

wants to merge 4 commits into from

Conversation

Hoikas
Copy link
Member

@Hoikas Hoikas commented Aug 20, 2018

Plasma stores its image data in plMipmap as BGR(A), with the exception of DXT formatted data. This changeset updates PRPShop to match the fixes in libHSPlasma in that regard. Also teaches PRPShop about PNG compressed images. I believe a more complete changeset from @Deledrius is incoming to improve the import/export image options.

Depends on:

@Deledrius
Copy link
Member

Correct. I'm working on a revamped dialog that should have a unified, flexible import/export.

@Hoikas Hoikas force-pushed the bgra branch 2 times, most recently from 1845165 to 6aa9853 Compare August 21, 2018 22:48
@Hoikas
Copy link
Member Author

Hoikas commented Aug 21, 2018

I updated the pull request to not change the RGB bitmask on the plDDSurface because this seemed to introduce an issue that could be triggered by importing a DDS file into a jpeg compressed image. Instead, we now flip the channels on import and export of dds data.

@Hoikas
Copy link
Member Author

Hoikas commented Mar 5, 2024

The CI failure here is due to H-uru/libhsplasma#127 being a prerequisite of this.

Co-authored-by: Michael Hansen <zrax0111@gmail.com>
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.

None yet

3 participants