search

xinxinlx / openjpeg

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

OpenJPEG writes to stderr #246

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
OpenJPEG has a nice system for people that don't want messages being logged to 
stdout/stderr. Unfortunatly this system wasn't used yet everywhere.

Attached patch replaces printf by opj_event_msg where possible (and removes 
some logs completely where it doesn't seem to make sense).

Patch was created using diff on OpenJPEG 2.0.0

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

Attachments:

GoogleCodeExporter commented 9 years ago 
Would it be possible for you to generate a unified diff (diff -ru) ? Your patch 
does not apply for me over here:

$ patch --verbose -p1 --dry-run < /tmp/test.patch 
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:39

GoogleCodeExporter commented 9 years ago 
see issue 184 for history

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

GoogleCodeExporter commented 9 years ago

Original comment by mathieu.malaterre on 25 Feb 2014 at 4:26

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 
Issue 184 has been merged into this issue.

Original comment by mathieu.malaterre on 7 Mar 2014 at 3:14

GoogleCodeExporter commented 9 years ago 
New patches created with commands
diff -ru Patched/src/lib/openjp2/image.c Original/src/lib/openjp2/image.c > 
Issue246-Patch2a.txt
diff -ru Patched/src/lib/openjp2/j2k.c Original/src/lib/openjp2/j2k.c > 
Issue247-Patch2b.txt
diff -ru Patched/src/lib/openjp2/openjpeg.c Original/src/lib/openjp2/openjpeg.c 
> Issue247-Patch2c.txt
diff -ru Patched/src/lib/openjp2/phix_manager.c 
Original/src/lib/openjp2/phix_manager.c > Issue247-Patch2d
diff -ru Patched/src/lib/openjp2/ppix_manager.c 
Original/src/lib/openjp2/ppix_manager.c > Issue247-Patch2e
diff -ru Patched/src/lib/openjp2/t2.c Original/src/lib/openjp2/t2.c > 
Issue247-Patch2f.txt
diff -ru Patched/src/lib/openjp2/t2.h Original/src/lib/openjp2/t2.h > 
Issue247-Patch2g.txt
diff -ru Patched/src/lib/openjp2/tcd.c Original/src/lib/openjp2/tcd.c > 
Issue247-Patch2h.txt
diff -ru Patched/src/lib/openjp2/tcd.h Original/src/lib/openjp2/tcd.h > 
Issue247-Patch2i.txt
diff -ru Patched/src/lib/openjp2/tgt.c Original/src/lib/openjp2/tgt.c > 
Issue247-Patch2j.txt
diff -ru Patched/src/lib/openjp2/tgt.h Original/src/lib/openjp2/tgt.h > 
Issue247-Patch2k.txt

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

Attachments:

GoogleCodeExporter commented 9 years ago

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