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

Remove redundant assignment (fixes compiler warning) #718

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

stweil
Copy link
Collaborator

@stweil stweil commented Oct 31, 2023

The member variable bpp is deprecated and a synonym for the member variable prec, which already has the same value assigned to it, so remove the redundant assignment to bpp. This fixes a compiler warning:

src/jp2kio.c: In function 'pixConvertToOpjImage':
src/jp2kio.c:726:9: warning: 'bpp' is deprecated: Use prec instead [-Wdeprecated-declarations]
726 | cmptparm[i].bpp = 8;
| ^~~~~~~~

The member variable bpp is deprecated and a synonym for the member variable prec,
which already has the same value assigned to it, so remove the redundant assignment
to bpp. This fixes a compiler warning:

src/jp2kio.c: In function 'pixConvertToOpjImage':
src/jp2kio.c:726:9: warning: 'bpp' is deprecated: Use prec instead [-Wdeprecated-declarations]
  726 |         cmptparm[i].bpp = 8;
      |         ^~~~~~~~

Signed-off-by: Stefan Weil <[email protected]>
@DanBloomberg DanBloomberg merged commit 6d2c1c5 into DanBloomberg:master Oct 31, 2023
10 of 11 checks passed
@DanBloomberg
Copy link
Owner

Thank you for this!

@stweil stweil deleted the fix_deprecated branch November 1, 2023 17:20
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