JWAT-9/JWAT-10/JWAT-13: New GZip multipart reader/writer

CR-JWAS-13 10

Keyboard shortcuts  
Summarize the review outcomes (optional)
 
#permalink

Details

Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author & Moderator 24m    
Reviewer - Complete 35m 10 Remove line break before else
Total   59m 10  
#permalink

Objectives

An new gzip reader and writer has been implemented.
The old gzipinputstream will be moved to test as a reference.
Part of JWAT-9, JWAT-10, JWAT-13.

Branches in review

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

There are no general comments on this review.
/jwat-common/.../common/DigestInputStreamNoSkip.java Changed
/jwat-common/.../common/MaxLengthRecordingInputStream.java Changed
/jwat-gzip/src/.../gzip/GzipConstants.java Changed 1
/jwat-gzip/src/.../gzip/GzipInputStream.java Changed
/jwat-gzip/src/.../jwat/gzip/GzipReader.java Changed 6
/jwat-gzip/src/.../gzip/GzipReaderEntry.java Changed
/jwat-gzip/src/.../jwat/gzip/GzipWriter.java Changed 3
/jwat-gzip/src/.../jwat/gzip/TestEncoding.java Changed
/jwat-gzip/src/.../jwat/gzip/TestFlagged.java Changed
/jwat-gzip/src/.../gzip/TestGzipReader.java Changed
/jwat-gzip/src/.../gzip/TestGzipWriter.java Changed
/jwat-gzip/.../gzip/TestGzipWriterCloning.java Changed
/jwat-gzip/src/.../gzip/TestInputStream.java Changed
/jwat-gzip/.../gzip/TestInputStreamReadSkip.java Added
Open in IDE #permalink
/jwat-gzip/src/.../jwat/gzip/TestInvalid.java Changed
/jwat-gzip/src/.../jwat/gzip/TestMagic.java Changed
/jwat-gzip/src/.../jwat/gzip/TestParams.java Changed
/jwat-gzip/src/.../gzip/TestSkippingClose.java Changed

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create issue

X
Assign To Me

Log time against