new String(..) in setSequenceSetId()


CR-JAL-213 3

Keyboard shortcuts  
  • Summarize the review outcomes (optional)


    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author 18m 1 It can be significant anywhere a string might be used as ...
    Reviewer - 100% reviewed 10m 2 This is the code in question. The sequenceSetId allows r...
    Reviewer - 0% reviewed 0m    
    Total   27m 3  


    Understand and identify if defensive copy for string is needed

    Branches in review


    Issues Raised From Comments

    Key Summary State Assignee

    General Comments

    There are no general comments on this review.
    /src/.../viewmodel/ Changed 3
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against