Laszlo Hornyak has uploaded a new change for review.

Change subject: engine: remove redundant check from setter in VmTemplate
......................................................................

engine: remove redundant check from setter in VmTemplate

checking if the status the same or the old value was null is not needed
in setters

Change-Id: I9fca59c0684afab7c7f1944ec1997b361257c6b0
Signed-off-by: Laszlo Hornyak <[email protected]>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java
1 file changed, 1 insertion(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/67/13367/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java
index a7e2a02..920b64c 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java
@@ -113,9 +113,7 @@
     }
 
     public void setstatus(VmTemplateStatus value) {
-        if (status != value) {
-            status = value;
-        }
+        status = value;
     }
 
     public String getvds_group_name() {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9fca59c0684afab7c7f1944ec1997b361257c6b0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to