Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Turn CSVRecord into a List #35

Closed
wants to merge 3 commits into from
Closed

Turn CSVRecord into a List #35

wants to merge 3 commits into from

Commits on Dec 3, 2018

  1. Turn CSVRecord into a List

    CSVRecord implements get(int) and size() methods so it’s already pretty
    much a list.  However, because it does not implement the List interface,
    users cannot take advantage of convenience methods such as subList nor
    can they pass records around to methods accepting lists.
    
    Make CSVRecord extend AbstractList so that it implements List interface.
    
    Because AbstractList pravides iterator() implementation, this allows us
    to delete said method (alongside toList() method) reducing amount of
    code.
    mina86 committed Dec 3, 2018
    Configuration menu
    Copy the full SHA
    9256af7 View commit details
    Browse the repository at this point in the history

Commits on Dec 12, 2018

  1. Configuration menu
    Copy the full SHA
    81c2dba View commit details
    Browse the repository at this point in the history

Commits on Dec 13, 2018

  1. document list is unmodifiable

    mina86 committed Dec 13, 2018
    Configuration menu
    Copy the full SHA
    5d43929 View commit details
    Browse the repository at this point in the history