Change measured_files to a set

Author: nedbat

coverage/control.py

@@ -683,15 +683,11 @@ def _get_file_reporters(self, morfs=None):
         if not morfs:
             morfs = self._data.measured_files()
 
-        # Be sure we have a list.
-        if not isinstance(morfs, (list, tuple)):
+        # Be sure we have a collection.
+        if not isinstance(morfs, (list, tuple, set)):
             morfs = [morfs]
 
-        file_reporters = []
-        for morf in morfs:
-            file_reporter = self._get_file_reporter(morf)
-            file_reporters.append(file_reporter)
-
+        file_reporters = [self._get_file_reporter(morf) for morf in morfs]
         return file_reporters
 
     def report(

coverage/data.py

@@ -252,8 +252,8 @@ def run_infos(self):
         return self._runs
 
     def measured_files(self):
-        """A list of all files that had been measured."""
-        return list(self._arcs or self._lines or {})
+        """A set of all files that had been measured."""
+        return set(self._arcs or self._lines or {})
 
     def __nonzero__(self):
         return bool(self._lines or self._arcs)

coverage/sqldata.py

@@ -304,7 +304,7 @@ def update(self, other_data, aliases=None):
         aliases = aliases or PathAliases()
 
         # See what we had already measured, for accurate conflict reporting.
-        this_measured = set(self.measured_files())
+        this_measured = self.measured_files()
 
         # lines
         if other_data._has_lines:
@@ -381,8 +381,8 @@ def has_arcs(self):
         return bool(self._has_arcs)
 
     def measured_files(self):
-        """A list of all files that had been measured."""
-        return list(self._file_map)
+        """A set of all files that had been measured."""
+        return set(self._file_map)
 
     def file_tracer(self, filename):
         """Get the plugin name of the file tracer for a file.