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

OpenJPEG doesn't compile on mac with gcc 4 #243

Closed
gcode-importer opened this issue Oct 8, 2013 · 6 comments
Closed

OpenJPEG doesn't compile on mac with gcc 4 #243

gcode-importer opened this issue Oct 8, 2013 · 6 comments

Comments

@gcode-importer
Copy link

Originally reported on Google Code with ID 243

Several methods are forward declared as static but the actual definition does not have
the static keyword (gcc4 with our build options chokes on this).

This was reported in OpenJPEG 2.0.0
MacOSX 10.7.5
Xcode 3.2.6

Attached patch is generated with diff.


Reported by brta.enfocus on 2013-10-08 12:29:48


- _Attachment: [fix gcc 4 compile errors](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-243/comment-0/fix gcc 4 compile errors)_
@gcode-importer
Copy link
Author

Please provide a unified diff thanks.

Reported by malaterre on 2014-02-24 08:43:31

@gcode-importer
Copy link
Author

Reported by malaterre on 2014-02-25 15:56:23

  • Labels added: Milestone-Release2.1

@gcode-importer
Copy link
Author

marking as wontfix until OP update status. thanks.

Reported by malaterre on 2014-03-03 11:53:36

  • Status changed: WontFix

@gcode-importer
Copy link
Author

New patch created with command
diff -ru Patched/src/lib/openjp2/t1.c Original/src/lib/openjp2/t1.c > Issue243-Patch2.txt

Reported by brta.enfocus on 2014-07-07 09:55:24


- _Attachment: [Issue243-Patch2.txt](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-243/comment-4/Issue243-Patch2.txt)_

@gcode-importer
Copy link
Author

Reported by detonin on 2014-07-08 11:44:26

  • Status changed: Accepted

@detonin detonin modified the milestone: OPJ v2.1.1 Jul 6, 2015
mayeut added a commit to mayeut/openjpeg that referenced this issue Jul 18, 2015
Still needs to check j2k.c & jp2.c
Update uclouvain#243
mayeut added a commit to mayeut/openjpeg that referenced this issue Jul 23, 2015
@mayeut
Copy link
Collaborator

mayeut commented Jul 23, 2015

Should be fixed but I can't verify this with my compiler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants