diff options
author | Olivier Grisel <olivier.grisel@ensta.org> | 2017-08-10 15:52:35 +0200 |
---|---|---|
committer | Olivier Grisel <olivier.grisel@ensta.org> | 2017-08-10 15:52:35 +0200 |
commit | dfdec3b7437374d9472cfb2d4322a18e797c3dfa (patch) | |
tree | ab10f1c695541cd2b6bb03376a3291dc17523002 /coverage/collector.py | |
parent | 1414d62424d6dc210609db7ee7eb18472d9a83f6 (diff) | |
download | python-coveragepy-dfdec3b7437374d9472cfb2d4322a18e797c3dfa.tar.gz |
FIX thread-safe Collector.save_data()
Diffstat (limited to 'coverage/collector.py')
-rw-r--r-- | coverage/collector.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/coverage/collector.py b/coverage/collector.py index cfdcf40..a318c10 100644 --- a/coverage/collector.py +++ b/coverage/collector.py @@ -374,7 +374,10 @@ class Collector(object): def abs_file_dict(d): """Return a dict like d, but with keys modified by `abs_file`.""" - return dict((abs_file(k), v) for k, v in iitems(d)) + # The call to list() ensures that the GIL protects the dictionary + # iterator against concurrent modifications by tracers running + # in other threads. + return dict((abs_file(k), v) for k, v in list(iitems(d))) if self.branch: covdata.add_arcs(abs_file_dict(self.data)) |