You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by bh...@apache.org on 2016/01/27 16:08:56 UTC

[2/2] git commit: updated refs/heads/master to a59cf68

Merge pull request #969 from rafaelweingartner/master-lrg-cs-hackday-002

Fixed return type Void to void in  DataMotionStrategy.The main changes are:
-	Changing methods Void to void.
-	Removal of the method Void copyAsync(DataObject srcData, DataObject
destData, AsyncCompletionCallback<CopyCommandResult> callback) that was
never used.
We noticed that methods form that class are using the return type Void
with capital V. This way that method has to return a null value at the
end.

Removed trim lines from XenServerStorageMotionStrategy.

The trim lines were removed from XenServerStorageMotionStrategy.

* pr/969:
  Changed return of methods from DataMotionStrategy, Void to void

Signed-off-by: Rohit Yadav <ro...@shapeblue.com>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/a59cf68f
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/a59cf68f
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/a59cf68f

Branch: refs/heads/master
Commit: a59cf68f5085ca69e3cdb57e254f0e236af1dec7
Parents: ba722f8 4a9487e
Author: Rohit Yadav <ro...@shapeblue.com>
Authored: Wed Jan 27 16:08:07 2016 +0100
Committer: Rohit Yadav <ro...@shapeblue.com>
Committed: Wed Jan 27 16:08:07 2016 +0100

----------------------------------------------------------------------
 .../api/storage/DataMotionStrategy.java         |  7 +--
 .../motion/AncientDataMotionStrategy.java       | 28 +++---------
 .../motion/StorageSystemDataMotionStrategy.java | 33 +++++---------
 .../storage/test/MockStorageMotionStrategy.java | 45 +-------------------
 .../motion/HypervStorageMotionStrategy.java     | 16 ++-----
 .../motion/SimulatorDataMotionStrategy.java     | 12 +-----
 .../motion/VmwareStorageMotionStrategy.java     | 19 ++-------
 .../motion/XenServerStorageMotionStrategy.java  | 19 ++-------
 8 files changed, 33 insertions(+), 146 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/a59cf68f/plugins/hypervisors/xenserver/src/org/apache/cloudstack/storage/motion/XenServerStorageMotionStrategy.java
----------------------------------------------------------------------