diff options
| author | Kevin Rocard <kevinx.rocard@intel.com> | 2013-06-11 11:37:03 +0200 |
|---|---|---|
| committer | David Wagner <david.wagner@intel.com> | 2014-02-12 17:04:01 +0100 |
| commit | a2d6452283924def8f9d46337ac7c2e2bcf6c352 (patch) | |
| tree | ae667f73c2f22650b18de3678c4e25def72ef9d2 /tools | |
| parent | 925c20b1889b4963c3890e9992385bb87d8fdb97 (diff) | |
| download | external_parameter-framework-a2d6452283924def8f9d46337ac7c2e2bcf6c352.zip external_parameter-framework-a2d6452283924def8f9d46337ac7c2e2bcf6c352.tar.gz external_parameter-framework-a2d6452283924def8f9d46337ac7c2e2bcf6c352.tar.bz2 | |
[coverage] Delete useless log message to gain performance
BZ: 115218
The __eq__ function is very used (1M time in my test),
but cProfile showed that it was one of the most slow
functions (in cumulate time).
Delete useless log in the function in order to gain
90% performance (10s to 1s in cumulate time).
Change-Id: I1ece15186cb30947f2c2af277cd0b326ea72c7ea
Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
Reviewed-on: http://android.intel.com:8080/112933
Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com>
Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com>
Reviewed-by: cactus <cactus@intel.com>
Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com>
Reviewed-by: buildbot <buildbot@intel.com>
Tested-by: buildbot <buildbot@intel.com>
Diffstat (limited to 'tools')
| -rwxr-xr-x | tools/coverage.py | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/tools/coverage.py b/tools/coverage.py index 7e7b801..0cd2a5d 100755 --- a/tools/coverage.py +++ b/tools/coverage.py @@ -76,12 +76,7 @@ class Element(): return "%s (%s)" % (self.name, self.tag) def __eq__(self, compared): - self.debug(lambda : "Comparing:\n%s" % self.dump()) - self.debug(lambda : "With:\n%s" % compared.dump()) - result = self.name == compared.name and self.children == compared.children - self.debug("Result is %s" % result) - return result - + return (self.name == compared.name) and (self.children == compared.children) def getName(self, default=""): return self.name or default |