diff options
| author | Kevin Rocard <kevinx.rocard@intel.com> | 2013-06-10 15:09:57 +0200 |
|---|---|---|
| committer | David Wagner <david.wagner@intel.com> | 2014-02-12 17:03:59 +0100 |
| commit | 33a4e33c51ce396851db36f07b869d6786c6e858 (patch) | |
| tree | 4b76c37891e076a493b8e4504c51b2d90f17d5c8 /tools | |
| parent | 7784f4a2a3b46bbeaf0f85f62630013861bbf3e5 (diff) | |
| download | external_parameter-framework-33a4e33c51ce396851db36f07b869d6786c6e858.zip external_parameter-framework-33a4e33c51ce396851db36f07b869d6786c6e858.tar.gz external_parameter-framework-33a4e33c51ce396851db36f07b869d6786c6e858.tar.bz2 | |
[Coverage] Delete dangerous assert, raise exception
BZ: 115218
An assert was checking function input parameters that came from an input
file. As a result an corrupted file would stop the program without the
opportunity to catch the error.
Remove the assert and let the function raise the appropriate exception.
Change-Id: Ic8d8d318ca191040c546e354c0911d56670b1030
Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
Reviewed-on: http://android.intel.com:8080/112709
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 | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/tools/coverage.py b/tools/coverage.py index c4250fa..6c4345c 100755 --- a/tools/coverage.py +++ b/tools/coverage.py @@ -244,8 +244,6 @@ class Element(): return self.nbUse > 0 def operationOnChild(self, path, operation): - # Stop if path is not consume yet but there is no child - assert(self.children or not path) if path: return self._operationPropagate(path, operation) |