Allon Mureinik has posted comments on this change.

Change subject: engine: Don't iterate maps with keySet()
......................................................................


Patch Set 3: Verified+1

1. Fixed variable names as per Roy's review
2. Added curly braces as per Tal's review
3. Replaced the entrySet() iteration in ImportVmModel with values(), which is 
more straight forward.
4. Changed commit message accordingly.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia17710ae5dcf61779cabb892638d3d8f6ac95d86
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to