Minor refactoring of API, unittests, etc.

Activity

CR-JWAS-28 8

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 12m 1 To add warnings to the writers diagnostics. Or at least d...
    Reviewer - Complete 12m 7 (2 defects) What is your intention here?
    Total   24m 8 (2 defects)  
    #permalink

    Objectives

    changeset: 188:ebb2ce1839cd
    user: nclarkekb
    date: Fri Dec 07 19:48:32 2012 +0100
    summary: Added some getters.

    changeset: 187:c208a58558ac
    user: nclarkekb
    date: Sun Oct 28 15:54:14 2012 +0100
    summary: Minor refactoring of API, unittests, etc.

    #permalink

    Summary

    JIRA issues created.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /jwat-arc/src/main/.../jwat/arc/ArcReader.java Changed
    /jwat-arc/src/.../arc/ArcReaderFactory.java Changed 1
    /jwat-common/src/.../common/HttpHeader.java Changed 1
    /jwat-common/.../common/PayloadWithHeaderAbstract.java Changed 1
    /jwat-common/src/.../common/UriProfile.java Changed
    /jwat-gzip/src/.../gzip/GzipTestHelper.java Changed
    /jwat-gzip/src/.../gzip/Test_Encoding.java Changed 1
    /jwat-warc/src/.../jwat/warc/WarcHeader.java Changed
    /jwat-warc/src/.../jwat/warc/WarcReader.java Changed
    /jwat-warc/src/.../jwat/warc/WarcWriter.java Changed 4
    /jwat-warc/src/.../warc/TestWarcRecord.java Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against