Two fixes so far:

1. Added CRC32Ext based on Adler32Ext - seemed to be missing when I updated my branch from HEAD, perhaps it was not in the original patch, or got missed.
2. Added defaults to GZipWriter#initialize to match open and Deflate#new:
    def initialize(io, level=nil, strategy=nil)
      @io = GZIPOutputStream.new(IOConverter.new(io).asOutputStream)
    end
Odd that this doesn't show up in any documentation.

Current issue is that in the above initialize method, the GZIPOutputStream.new is being passed a RubyObject, not an OutputStream. Looks like a JRuby issue.

Tom and I agree we can go forward with the non-security-friendly checksumming for now, but we'll probably need to look at another option (like just implementing CRC32 and Adler32 ourselves).

--
Charles Oliver Nutter @ headius.blogspot.com
JRuby Developer @ jruby.sourceforge.net
Application Architect @ www.ventera.com

Reply via email to