Change in vdsm[master]: vdsm: Remove unreachable code

mbetak at redhat.com mbetak at redhat.com
Tue Feb 2 16:12:33 UTC 2016


Martin Betak has uploaded a new change for review.

Change subject: vdsm: Remove unreachable code
......................................................................

vdsm: Remove unreachable code

Change-Id: Ie84749d01a735bc8177c7d16fb0612636f547004
Signed-off-by: Martin Betak <mbetak at redhat.com>
---
M lib/vdsm/utils.py
M tests/schemaValidationTest.py
M vdsm/API.py
3 files changed, 0 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/97/52997/1

diff --git a/lib/vdsm/utils.py b/lib/vdsm/utils.py
index 3863d2b..13a8bc1 100644
--- a/lib/vdsm/utils.py
+++ b/lib/vdsm/utils.py
@@ -295,7 +295,6 @@
             if e.args[0] == os.errno.EINTR:
                 continue
             raise
-        break
 
 
 class CommandStream(object):
diff --git a/tests/schemaValidationTest.py b/tests/schemaValidationTest.py
index 97fcd86..601aa5a 100644
--- a/tests/schemaValidationTest.py
+++ b/tests/schemaValidationTest.py
@@ -111,7 +111,6 @@
                 except KeyError:
                     raise AssertionError('Missing method %s.%s' % (
                                          spec_class, method_name))
-                    continue
 
                 # inspect apiobj and gather args and default args
                 args = ctorArgs + self._get_args(method_obj)
diff --git a/vdsm/API.py b/vdsm/API.py
index adb20a2..d23b09d 100644
--- a/vdsm/API.py
+++ b/vdsm/API.py
@@ -256,7 +256,6 @@
                                        'message': 'Failed to create '
                                                   'sysprep floppy image. '
                                                   'No space on /tmp?'}}
-                    return errCode['createErr']
 
             if not graphics.isSupportedDisplayType(vmParams):
                 return {'status': {'code': errCode['createErr']


-- 
To view, visit https://gerrit.ovirt.org/52997
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie84749d01a735bc8177c7d16fb0612636f547004
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak at redhat.com>


More information about the vdsm-patches mailing list