diff options
| author | Gabriel Feceoru <gabriel.feceoru@intel.com> | 2016-03-02 15:49:21 +0200 |
|---|---|---|
| committer | Dylan Baker <baker.dylan.c@gmail.com> | 2016-03-16 09:36:16 -0700 |
| commit | c1db34add3b68090f8ed2b95b18c82f37b24dc63 (patch) | |
| tree | dab563caacccb9e8de33b3c24bbc3ed59ad61804 /framework/summary | |
| parent | 05cb199e5ed5a4e9d0fd559be8dafcb6982ad629 (diff) | |
framework: Replace deepcopy in summary feature
Deepcopy seems to be very slow and, for specific profiles (igt), causes errors.
Replacing it with update() method fixes the above problems.
v2 (Dylan):
- remove copy import
Signed-off-by: Gabriel Feceoru <gabriel.feceoru@intel.com>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Diffstat (limited to 'framework/summary')
| -rw-r--r-- | framework/summary/feature.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/framework/summary/feature.py b/framework/summary/feature.py index 3bec618f7..cd903c2ee 100644 --- a/framework/summary/feature.py +++ b/framework/summary/feature.py @@ -25,8 +25,6 @@ from __future__ import ( absolute_import, division, print_function, unicode_literals ) -import copy - try: import simplejson as json except ImportError: @@ -68,7 +66,8 @@ class FeatResults(object): # pylint: disable=too-few-public-methods options.OPTIONS.exclude_filter = exclude_filter options.OPTIONS.include_filter = include_filter - profiles[feature] = copy.deepcopy(profile_orig) + profiles[feature] = profile.TestProfile() + profiles[feature].update(profile_orig) # An empty list will raise PiglitFatalError exception # But for reporting we need to handle this situation |