Alissa Bonas has uploaded a new change for review.

Change subject: core: clean javadoc in VmHandler
......................................................................

core: clean javadoc in VmHandler

Remove unused parameters in javadoc and rename ones incorrectly named
in VMHandler class

Change-Id: I321fa2a4e1dc10c0ef95e05bfadbe19242ec77e2
Signed-off-by: Alissa Bonas <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
1 file changed, 6 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/83/10283/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
index e584e6f..25d8774 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
@@ -165,8 +165,6 @@
     /**
      * Lock VM after check its status, If VM status is locked, we throw an 
exception.
      *
-     * @param status
-     *            - The status of the VM
      * @param vmId
      *            - The ID of the VM.
      */
@@ -179,8 +177,6 @@
     /**
      * Lock VM with compensation, after checking its status, If VM status is 
locked, we throw an exception.
      *
-     * @param status
-     *            - The status of the VM
      * @param vmId
      *            - The ID of the VM, which we want to lock.
      * @param compensationContext
@@ -375,7 +371,7 @@
      *     Display type : Spice or Vnc
      * @param numOfMonitors
      *     Number of monitors
-     * @param messages
+     * @param reasons
      *     Messages for CanDoAction().
      * @return
      */
@@ -415,10 +411,11 @@
     /**
      * Checks that the USB policy is legal for the VM. If it is ENABLED_NATIVE 
then it is legal only
      * in case the cluster level is >= 3.1. If it is ENABLED_LEGACY then it is 
not legal on Linux VMs.
-     * @param vm
-     *     The VM object
-     * @param messages
-     *     Messages for CanDoAction().
+     *
+     * @param usbPolicy
+     * @param osType
+     * @param vdsGroup
+     * @param messages - Messages for CanDoAction()
      * @return
      */
     public static boolean isUsbPolicyLegal(UsbPolicy usbPolicy,


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

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

Reply via email to