Change in vdsm[master]: Make utils pep8 compliant

zhshzhou at linux.vnet.ibm.com zhshzhou at linux.vnet.ibm.com
Wed Nov 7 05:41:36 UTC 2012


Zhou Zheng Sheng has posted comments on this change.

Change subject: Make utils pep8 compliant
......................................................................


Patch Set 1: Verified; Looks good to me, but someone else must approve

Pass autobuild.sh. pydiff finds some changes, the first one splitting import into separate ones in line 31 vs 31, the second one is introducing a temp variable in line 492 vs 519, the last one is changing "== "to "is" when comparing identity to "None" in line 656 vs 689. All changes are acceptable.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I514660d73071f8fb417fea0cc54973a55002992f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list