Skip to content
This repository has been archived by the owner on Dec 31, 2020. It is now read-only.

Restore 2 0 8 abi #275

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

Conversation

StefanBruens
Copy link
Contributor

This fixes all but one issue in #259

Only remaining incompatibility is the added virtual method OutputHandler::endImage()

Although adding a method argument with default argument keeps API
compatibility, it changes the function type.

Use function overloading instead and implement the old one as a wrapper
to the extended one, supplying the default argument.

Fixes part of castano#259

Signed-off-by: Stefan Brüns <[email protected]>
The linear transform has never been implemented, so just add as a stub
for API/ABI compatibility.

Fixes part of castano#259

Signed-off-by: Stefan Brüns <[email protected]>
The only colortransform actually implemented was the transform from
RGB to YCoCG (ColorTransform_YCoCg).
Restore the full enum for API compatibility, and allow the transform
to be specified using InputOptions::setColortransform.

Fixes part of castano#259

Signed-off-by: Stefan Brüns <[email protected]>
@StefanBruens
Copy link
Contributor Author

Failed check is unrelated.
Apparently MSVC makes the first 2 arguments to compress_etc1 const, and then can not find a matching implementation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant