Moti Asayag has submitted this change and it was merged.
Change subject: core: Cleanup SetupNetworksHelper part 4
......................................................................
core: Cleanup SetupNetworksHelper part 4
1. Added some blank lines to separate bulks of code.
2. Reduced visibility of methods as possible.
3. Removed some double ifs.
4. Added doc of return value to validate() method.
Change-Id: Id36c8e3a25530c134b0a279692814f5efa54a3ed
Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SetupNetworksHelper.java
1 file changed, 26 insertions(+), 22 deletions(-)
Approvals:
Moti Asayag: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/4600
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id36c8e3a25530c134b0a279692814f5efa54a3ed
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>