Change in vdsm[master]: pep8: Fix false negative whitespace error

alitke at redhat.com alitke at redhat.com
Tue May 20 20:38:20 UTC 2014


Adam Litke has uploaded a new change for review.

Change subject: pep8: Fix false negative whitespace error
......................................................................

pep8: Fix false negative whitespace error

In newer versions of pep8, I get the following error:

./vdsm/virt/migration.py:223:19: E225 missing whitespace around operator

This is a false negative which is breaking the build.  We can work
around the issue by enclosing the first -1 in ().  For consistency, I've
enclosed the second one as well.

pep8 version: 1.5.4

Change-Id: I07f2268f8dfc385589b6f6d9531b0d4ba2131323
Signed-off-by: Adam Litke <alitke at redhat.com>
---
M vdsm/virt/migration.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/48/27948/1

diff --git a/vdsm/virt/migration.py b/vdsm/virt/migration.py
index 8f58d82..da7fad0 100644
--- a/vdsm/virt/migration.py
+++ b/vdsm/virt/migration.py
@@ -220,7 +220,7 @@
                  'operation aborted',                # error message
                  libvirt.VIR_ERR_WARNING,            # error level
                  '', '', '',                         # str1, str2, str3,
-                 -1, -1)                             # int1, int2
+                 (-1), (-1))                         # int1, int2
         raise e
 
     def run(self):


-- 
To view, visit http://gerrit.ovirt.org/27948
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I07f2268f8dfc385589b6f6d9531b0d4ba2131323
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke at redhat.com>


More information about the vdsm-patches mailing list