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

[libpng18] [code cleanup] Remove 10700 checkes where possible. #611

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

jbowler
Copy link
Contributor

@jbowler jbowler commented Oct 3, 2024

This removes certain of the 10700 checks which can safely be removed as
they depend on the major require in libpng-1.7 which is not happening
in libpng-1.8.

The checks in pngvalid.c remain because it may (should) be possible to
fix or partially the underlying libpng bugs in 1.8. The checks document
what those bugs are and ideally the 10700 branch of the #if's can be
used after a fix.

Signed-off-by: John Bowler [email protected]

This removes certain of the 10700 checks which can safely be removed as
they depend on the major require in libpng-1.7 which is not happening
in libpng-1.8.

The checks in pngvalid.c remain because it may (should) be possible to
fix or partially the underlying libpng bugs in 1.8.  The checks document
what those bugs are and ideally the 10700 branch of the #if's can be
used after a fix.

Signed-off-by: John Bowler <[email protected]>
@jbowler jbowler changed the title code cleanup: Remove 10700 checkes where possible. [libpng18] [code cleanup] Remove 10700 checkes where possible. Oct 3, 2024
@ctruta ctruta merged commit bea7fde into pnggroup:libpng18 Oct 9, 2024
1 check passed
@jbowler jbowler deleted the libpng18-v1.7-cleanup branch October 9, 2024 15:06
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