Skip Menu |

This queue is for tickets about the Compress-Zlib CPAN distribution.

Report information
The Basics
Id: 36268
Status: resolved
Priority: 0/
Queue: Compress-Zlib

People
Owner: Nobody in particular
Requestors: cpan [...] pjedwards.co.uk
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in:
  • 2.005
  • 2.006
  • 2.007
  • 2.008
  • 2.009
  • 2.010
  • 2.011
Fixed in: (no value)



Subject: Question about wide character use in memGzip between version 2.004 and 2.005
Hello and thanks for Compress::Zlib. I am upgrading from Compress::Zlib from 1.41 to 2.009 and as part of the upgrade I ran 1.41's tests against 2.009, I noticed a change in the behaviour of IO::Compress::memGzip and how it treats wide characters. Prior to 2.005 IO::Compress::memGzip did not have: $] >= 5.008 and (utf8::downgrade($$string, 1) or croak "Wide character in memGzip"); and the encoding test also had: my $s = "\x{df}\x{100}"; my $cs = Compress::Zlib::memGzip($s); I did read the Changes file for Compress::Zlib but this change was not mentioned, I also read the docs and the FAQ. I did find in the Changes file for IO::Compress::Zlib 2.005 18 June 2007 * IO::Compress::Gzip & IO::Uncompress::Gunzip - RFC1952 says that the FNAME & FCOMMENT header fields must be ISO 8859-1 (LATIN-1) characters. The code can optionally police this. Added a fix for this logic when running on EBCDIC. This explains why the wide char check was added to memGunzip (I think). Just wondering if you could shed some light on the change to memGzip? Many thanks, Peter (Stig) Edwards
Hi Peter, So the reason for the new code is partially down to the rewrite I did in the code. Previously some of this code was dealt with in C code. A lot of that code migrated to straight Perl in version 2.0 Are you having a problem with the new logic, or were you just curious? Paul
Thanks for the reply. I'm not having a problem with the new logic, I'm just curious. I don't anticipate any problems with the new logic. I'll be upgrading some more instances of Compress::Zlib to 2.009 today. memGzip() is one of the more often used subroutines and I just wanted to check the change in behaviour was deliberate. Cheers, Peter (Stig) Edwards
No problem, I'll close this ticket then. If you find any problems when you upgrade, please raise another ticket. cheers Paul