Bug #78654 for Image-MetaData-JPEG: Found/Fixed an issue with Image::MetaData::JPEG

Sun Jul 29 22:17:54 2012 keng [...] apple.com - Ticket created

Subject:	Found/Fixed an issue with Image::MetaData::JPEG
Date:	Sun, 29 Jul 2012 19:17:35 -0700
To:	bug-Image-MetaData-JPEG [...] rt.cpan.org
From:	Ken Greenebaum <keng [...] apple.com>

I am running Perl v5.12.4 on a Mac running OS X 10.8 and encountered an issue with Image::MetaData::JPEG where even in the most simple re-write case a spurious 0x20 is inserted after every marker written causing the resulting jpeg file to be corrupt. The issue might not be in the library, but in my perl program hosting the library however the library should ideally be robust to this. Simple example: my $image = new Image::MetaData::JPEG('source.jpg', $parseSegments); $image->save('rewrite.jpg'); I found that replacing the comma enumeration with a period concatenation in the return from the module's Segment.pm:output_segment_data() fixed the issue for me: return print {$out} $preamble.$this->data(0, $length); I hope this can help someone else. -Ken

Sat Nov 16 11:10:20 2013 bettelli [...] cpan.org - Correspondence added

Hi, thank you for your remark and excuse me for the long delay. I suspect the problem is due to an interaction between your perl program and my library. You (or your default setup) have probably set the "output field separator" variable (that is $, ) to a space instead of leaving it undef as usual. Collapsing the elements of the list toghether with the dot operator of course circumvents the problem, since list items are joined before they are output, thus making the output field separator irrelevant. Show quoted text

>> however the library should ideally be robust to this.

OK, I will adopt this suggestion in the next bug fix release. Thank you again, regards, S. Bettelli

Sat Nov 16 11:10:20 2013 The RT System itself - Status changed from 'new' to 'open'

Sat Nov 16 11:18:30 2013 bettelli [...] cpan.org - Severity Important added

Sat Nov 16 11:18:30 2013 bettelli [...] cpan.org - Broken in 0.153 added

Tue Feb 04 22:12:37 2014 rjbs [...] cpan.org - Correspondence added

Thanks, fixed in 0.155 -- rjbs

Tue Feb 04 22:12:38 2014 rjbs [...] cpan.org - Status changed from 'open' to 'resolved'

Wed Feb 05 14:51:12 2014 bettelli [...] cpan.org - Fixed in 0.155 added