FtpServer
  1. FtpServer
  2. FTPSERVER-49

Specific return type for UserManager.getAllUserNames() not documented

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      UserManager.getAllUserNames() is declared as returning a Collection, but the comments do not specify that it is a collection of Strings. This is partially implied by the name of the method, but it would be helpful to explicitly specify this until the decision is made to exploit Java 1.5 generics in the source code.

        Activity

        Clinton Foster created issue -
        Hide
        Niklas Gustavsson added a comment -

        I've changed the method signature so that it instead returns an array of Strings. This will make the return value strongly typed without the need for generics at this point.

        Change is rev 467470.

        Show
        Niklas Gustavsson added a comment - I've changed the method signature so that it instead returns an array of Strings. This will make the return value strongly typed without the need for generics at this point. Change is rev 467470.
        Niklas Gustavsson made changes -
        Field Original Value New Value
        Assignee Niklas Gustavsson [ niklas ]
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Niklas Gustavsson made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        27m 53s 1 Niklas Gustavsson 24/Oct/06 21:41
        Resolved Resolved Closed Closed
        343d 27m 1 Niklas Gustavsson 02/Oct/07 22:09

          People

          • Assignee:
            Niklas Gustavsson
            Reporter:
            Clinton Foster
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development