Skip to content

Fixed signedness comparison warning#7203

Merged
hugovk merged 1 commit intopython-pillow:mainfrom
radarhere:numresolution
Jun 8, 2023
Merged

Fixed signedness comparison warning#7203
hugovk merged 1 commit intopython-pillow:mainfrom
radarhere:numresolution

Conversation

@radarhere
Copy link
Member

https://github.com/python-pillow/Pillow/actions/runs/5186598166/jobs/9347921664#step:7:118

src/libImaging/Jpeg2KEncode.c: In function ‘j2k_encode_entry’:
src/libImaging/Jpeg2KEncode.c:467:27: warning: comparison of integer expressions of different signedness: ‘unsigned int’ and ‘int’ [-Wsign-compare]
467 | while (tile_width < (1 << (params.numresolution - 1U)) || tile_height < (1 << (params.numresolution - 1U))) {
| ^
src/libImaging/Jpeg2KEncode.c:467:79: warning: comparison of integer expressions of different signedness: ‘unsigned int’ and ‘int’ [-Wsign-compare]
467 | while (tile_width < (1 << (params.numresolution - 1U)) || tile_height < (1 << (params.numresolution - 1U))) {
| ^

@hugovk hugovk merged commit e3cca42 into python-pillow:main Jun 8, 2023
@radarhere radarhere deleted the numresolution branch June 8, 2023 09:28
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