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

parameterize box test #2792

Merged
merged 4 commits into from
Sep 14, 2023
Merged

parameterize box test #2792

merged 4 commits into from
Sep 14, 2023

Conversation

mo271
Copy link
Member

@mo271 mo271 commented Sep 13, 2023

This is factored out, in order to make #2791 smaller and easier to review.
The diff only looks large, because we remove the outer loop
for (int compress_box = 0; compress_box <= 1; ++compress_box) {...}

@mo271 mo271 requested a review from szabadka September 13, 2023 14:50
@mo271 mo271 enabled auto-merge (squash) September 13, 2023 14:51
lib/jxl/encode_test.cc Outdated Show resolved Hide resolved
@mo271 mo271 merged commit 97e5ab0 into libjxl:main Sep 14, 2023
92 checks passed
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

2 participants