Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0
    • Component/s: None
    • Labels:
      None

      Description

      There is unnecessary work in ExtendedBufferedReader.

      It offers:

      skip(long)
      skipUntil(char)
      readUntil(char)

      none are ever called. skip(long) is required to fulfill the superclass, but could be stubbed out as it's never called.

      The class is also unnecessarily public for CSV's uses.

      1. SANDBOX-219.patch
        6 kB
        Henri Yandell

        Activity

        Hide
        jacopoc Jacopo Cappellato added a comment -

        Patch committed in rev. 1065948

        Show
        jacopoc Jacopo Cappellato added a comment - Patch committed in rev. 1065948
        Hide
        jacopoc Jacopo Cappellato added a comment -

        Is it ok if I commit Henri's patch?

        Show
        jacopoc Jacopo Cappellato added a comment - Is it ok if I commit Henri's patch?
        Hide
        jacopoc Jacopo Cappellato added a comment -

        I have tested this code and it still works well with trunk.

        Show
        jacopoc Jacopo Cappellato added a comment - I have tested this code and it still works well with trunk.
        Hide
        bayard Henri Yandell added a comment -

        Patch that strips out the unnecessary code. Class has already been made package private.

        Show
        bayard Henri Yandell added a comment - Patch that strips out the unnecessary code. Class has already been made package private.

          People

          • Assignee:
            Unassigned
            Reporter:
            bayard Henri Yandell
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development