xinxinlx / openjpeg

Automatically exported from code.google.com/p/openjpeg
Other
0 stars 0 forks source link

building 64bit version has lots of warnings #244

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
If the build target is 64bit, OPJ_SIZE_T is also 64bit. This value is often 
assigned to a OPJ_UINT32 which causes compile warnings.

Attached is a diff patch that cleans up those warnings by adding an explicit 
cast. 

Original issue reported on code.google.com by brta.enf...@gmail.com on 8 Oct 2013 at 12:33

GoogleCodeExporter commented 9 years ago
Added attachment mentioned in description.

Original comment by brta.enf...@gmail.com on 8 Oct 2013 at 12:34

Attachments:

GoogleCodeExporter commented 9 years ago
Could you please provide a unified diff please ?

$ patch --verbose -p1 --dry-run < /tmp/remove\ compiler\ warnings 
Hmm...patch unexpectedly ends in middle of line
  I can't seem to find a patch in there anywhere.
patch: **** Only garbage was found in the patch input.

Original comment by mathieu.malaterre on 24 Feb 2014 at 8:41

GoogleCodeExporter commented 9 years ago

Original comment by mathieu.malaterre on 25 Feb 2014 at 3:56

GoogleCodeExporter commented 9 years ago
marking as wontfix until OP update status. thanks.

Original comment by mathieu.malaterre on 3 Mar 2014 at 11:53

GoogleCodeExporter commented 9 years ago
New patches created with commands
diff -ru Patched/src/lib/openjp2/image.c Original/src/lib/openjp2/image.c > 
Issue244-Patch2.txt
diff -ru Patched/src/lib/openjp2/j2k.c Original/src/lib/openjp2/j2k.c > 
Issue244-Patch2b.txt
diff -ru Patched/src/lib/openjp2/jp2.c Original/src/lib/openjp2/jp2.c > 
Issue244-Patch2c.txt
diff -ru Patched/src/lib/openjp2/mqc.c Original/src/lib/openjp2/mqc.c > 
Issue244-Patch2d.txt
diff -ur diff -r Patched/src/lib/openjp2/raw.c Original/src/lib/openjp2/raw.c
diff -ru Patched/src/lib/openjp2/raw.c Original/src/lib/openjp2/raw.c > 
Issue244-Patch2e.txt
diff -ru Patched/src/lib/openjp2/t2.c Original/src/lib/openjp2/t2.c > 
Issue244-Patch2f.txt

Original comment by brta.enf...@gmail.com on 7 Jul 2014 at 10:02

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by antonin on 8 Jul 2014 at 11:43