Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove use of deprecated OpenJPEG "bpp" member #7090

Merged
merged 1 commit into from
Apr 16, 2023

Conversation

radarhere
Copy link
Member

https://github.com/python-pillow/Pillow/actions/runs/4696665571/jobs/8326957733#step:6:116

src/libImaging/Jpeg2KEncode.c: In function ‘j2k_encode_entry’:
src/libImaging/Jpeg2KEncode.c:345:9: warning: ‘bpp’ is deprecated: Use prec instead [-Wdeprecated-declarations]
345 | image_params[n].bpp = bpp;

We are already using prec just before this, so bpp can just be removed.

image_params[n].prec = prec;
image_params[n].bpp = bpp;

See uclouvain/openjpeg#1383 for more information.

@hugovk hugovk added the automerge Automatically merge PRs that are ready label Apr 16, 2023
@mergify mergify bot merged commit aec7a8d into python-pillow:main Apr 16, 2023
@radarhere radarhere deleted the openjpeg branch April 16, 2023 21:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
automerge Automatically merge PRs that are ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants