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

unsigned int16 on Solaris 11.2/sparc #796

Closed
joernclausen opened this issue Jul 13, 2016 · 2 comments
Closed

unsigned int16 on Solaris 11.2/sparc #796

joernclausen opened this issue Jul 13, 2016 · 2 comments
Milestone

Comments

@joernclausen
Copy link

Compiling openjpeg on Solaris 11.2/sparc using GCC 4.7.5 fails on "u_int16_t" in src/bin/jp2/convert.c. Changing this to "uint16_t" seems to fix the problem.

@mayeut mayeut added this to the OPJ v2.1.1 milestone Jul 13, 2016
@mayeut
Copy link
Collaborator

mayeut commented Jul 13, 2016

This is already fixed in release 2.1.1 & master

@mayeut mayeut closed this as completed Jul 13, 2016
stweil added a commit to stweil/openjpeg that referenced this issue Jul 14, 2016
The type casts which used this data type can be removed by changing
the signature of function swap16. As this function is called with
unsigned variables, this change is reasonable.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
@stweil
Copy link
Contributor

stweil commented Jul 14, 2016

It was only fixed for jp2, but is still open for jpwl. I sent a PR which fixes it there, too.

mayeut pushed a commit that referenced this issue Jul 14, 2016
The type casts which used this data type can be removed by changing
the signature of function swap16. As this function is called with
unsigned variables, this change is reasonable.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
malaterre pushed a commit that referenced this issue Sep 13, 2016
The type casts which used this data type can be removed by changing
the signature of function swap16. As this function is called with
unsigned variables, this change is reasonable.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
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

No branches or pull requests

3 participants