Uploaded image for project: 'Commons IO'
  1. Commons IO
  2. IO-230

DirectoryWalker needs to use generics to avoid compilation warnings

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: 2.0
    • Component/s: Utilities
    • Labels:

      Description

      When overriding methods such as handleFile, a generic Collection object is passed in argument. One is expected to add something to it, such as in this excerpt from DirectoryWalker's own javadoc:

        protected void handleFile(File file, int depth, Collection results) {
          [...]
          results.add(file);  // This lines triggers a warning.
        }
      

      This code triggers a warning: [unchecked] unchecked call to add(E) as a member of the raw type java.util.Collection. I believe the right fix is to change DirectoryWalker to be parameterized by a type T so that the signature of handleFile (and other similar methods) can be changed to receive a Collection<T> in argument.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              tsuna Benoit Sigoure
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: