You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by ga...@apache.org on 2016/10/23 13:43:42 UTC
[1/3] jclouds git commit: Make inner class static
Repository: jclouds
Updated Branches:
refs/heads/master 7e21cf0a4 -> d1508b2d2
Make inner class static
Found via error-prone.
Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/d1508b2d
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/d1508b2d
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/d1508b2d
Branch: refs/heads/master
Commit: d1508b2d231952a8930939daaadbd50b7fc398cd
Parents: 25baad6
Author: Andrew Gaul <ga...@apache.org>
Authored: Sun Oct 23 06:23:02 2016 -0700
Committer: Andrew Gaul <ga...@apache.org>
Committed: Sun Oct 23 06:39:46 2016 -0700
----------------------------------------------------------------------
.../v1/blobstore/RegionScopedSwiftBlobStoreParallelLiveTest.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/jclouds/blob/d1508b2d/apis/openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedSwiftBlobStoreParallelLiveTest.java
----------------------------------------------------------------------
diff --git a/apis/openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedSwiftBlobStoreParallelLiveTest.java b/apis/openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedSwiftBlobStoreParallelLiveTest.java
index 815f15e..50e3ea9 100644
--- a/apis/openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedSwiftBlobStoreParallelLiveTest.java
+++ b/apis/openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedSwiftBlobStoreParallelLiveTest.java
@@ -173,7 +173,7 @@ public class RegionScopedSwiftBlobStoreParallelLiveTest extends BaseBlobStoreInt
}
}
- private final class RandomFileWriter implements Runnable {
+ private static final class RandomFileWriter implements Runnable {
private final RandomAccessFile raf;
private final long begin;
private final long end;
[3/3] jclouds git commit: Remove unused imports
Posted by ga...@apache.org.
Remove unused imports
Found via error-prone.
Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/2a04a52d
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/2a04a52d
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/2a04a52d
Branch: refs/heads/master
Commit: 2a04a52d568625148c3067852ca51d78deecc1d8
Parents: 7e21cf0
Author: Andrew Gaul <ga...@apache.org>
Authored: Mon Jun 6 18:26:00 2016 -0700
Committer: Andrew Gaul <ga...@apache.org>
Committed: Sun Oct 23 06:39:46 2016 -0700
----------------------------------------------------------------------
apis/docker/src/main/java/org/jclouds/docker/domain/Config.java | 1 -
.../compute/GoogleComputeEngineServiceAdapter.java | 1 -
.../jclouds/googlecomputeengine/options/ImageCreationOptions.java | 1 -
3 files changed, 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/jclouds/blob/2a04a52d/apis/docker/src/main/java/org/jclouds/docker/domain/Config.java
----------------------------------------------------------------------
diff --git a/apis/docker/src/main/java/org/jclouds/docker/domain/Config.java b/apis/docker/src/main/java/org/jclouds/docker/domain/Config.java
index 58658de..abba227 100644
--- a/apis/docker/src/main/java/org/jclouds/docker/domain/Config.java
+++ b/apis/docker/src/main/java/org/jclouds/docker/domain/Config.java
@@ -23,7 +23,6 @@ import static org.jclouds.docker.internal.NullSafeCopies.copyWithNullOf;
import java.util.List;
import java.util.Map;
-import org.jclouds.docker.domain.HostConfig.Builder;
import org.jclouds.javax.annotation.Nullable;
import org.jclouds.json.SerializedNames;
http://git-wip-us.apache.org/repos/asf/jclouds/blob/2a04a52d/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java
----------------------------------------------------------------------
diff --git a/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java b/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java
index 07ee1f3..912f554 100644
--- a/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java
+++ b/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java
@@ -34,7 +34,6 @@ import javax.inject.Inject;
import javax.inject.Named;
import org.jclouds.compute.ComputeServiceAdapter;
-import org.jclouds.compute.domain.Hardware;
import org.jclouds.compute.domain.NodeMetadata;
import org.jclouds.compute.domain.OsFamily;
import org.jclouds.compute.domain.Template;
http://git-wip-us.apache.org/repos/asf/jclouds/blob/2a04a52d/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/options/ImageCreationOptions.java
----------------------------------------------------------------------
diff --git a/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/options/ImageCreationOptions.java b/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/options/ImageCreationOptions.java
index 0a33ada..58ff6c9 100644
--- a/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/options/ImageCreationOptions.java
+++ b/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/options/ImageCreationOptions.java
@@ -19,7 +19,6 @@ package org.jclouds.googlecomputeengine.options;
import static com.google.common.base.Preconditions.checkNotNull;
import org.jclouds.googlecomputeengine.domain.Image.RawDisk;
-import org.jclouds.googlecomputeengine.domain.NewTargetInstance.Builder;
import org.jclouds.javax.annotation.Nullable;
import org.jclouds.json.SerializedNames;
import org.jclouds.googlecomputeengine.domain.Deprecated;
[2/3] jclouds git commit: Address confusing operator precedence
Posted by ga...@apache.org.
Address confusing operator precedence
Found via error-prone.
Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/25baad6d
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/25baad6d
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/25baad6d
Branch: refs/heads/master
Commit: 25baad6d324266ece33eb60a76127407afd38536
Parents: 2a04a52
Author: Andrew Gaul <ga...@apache.org>
Authored: Fri Jul 15 19:05:08 2016 -0700
Committer: Andrew Gaul <ga...@apache.org>
Committed: Sun Oct 23 06:39:46 2016 -0700
----------------------------------------------------------------------
.../java/org/jclouds/ovf/xml/internal/BaseEnvelopeHandler.java | 4 ++--
.../org/jclouds/compute/StubComputeServiceIntegrationTest.java | 4 ++--
...airPlacementAndSecurityGroupsAsNeededAndReturnRunOptions.java | 4 ++--
.../compute/GoogleComputeEngineServiceAdapter.java | 2 +-
4 files changed, 7 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/jclouds/blob/25baad6d/compute/src/main/java/org/jclouds/ovf/xml/internal/BaseEnvelopeHandler.java
----------------------------------------------------------------------
diff --git a/compute/src/main/java/org/jclouds/ovf/xml/internal/BaseEnvelopeHandler.java b/compute/src/main/java/org/jclouds/ovf/xml/internal/BaseEnvelopeHandler.java
index 69f850a..db1f100 100644
--- a/compute/src/main/java/org/jclouds/ovf/xml/internal/BaseEnvelopeHandler.java
+++ b/compute/src/main/java/org/jclouds/ovf/xml/internal/BaseEnvelopeHandler.java
@@ -93,8 +93,8 @@ public class BaseEnvelopeHandler<V extends BaseVirtualSystem<V>, B extends BaseV
inNetwork = true;
} else if (equalsOrSuffix(qName, "VirtualSystem")) {
inVirtualSystem = true;
- } else if (extensionHandlers.containsKey(qName) || attributes.containsKey("type")
- && extensionHandlers.containsKey(attributes.get("type"))) {
+ } else if (extensionHandlers.containsKey(qName) || (attributes.containsKey("type")
+ && extensionHandlers.containsKey(attributes.get("type")))) {
inExtensionSection = true;
extensionHandler = extensionHandlers.get(qName).get();
} else if (qName.endsWith("Section")) {
http://git-wip-us.apache.org/repos/asf/jclouds/blob/25baad6d/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java
----------------------------------------------------------------------
diff --git a/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java b/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java
index 9c546f4..cfd904e 100644
--- a/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java
+++ b/compute/src/test/java/org/jclouds/compute/StubComputeServiceIntegrationTest.java
@@ -418,8 +418,8 @@ public class StubComputeServiceIntegrationTest extends BaseComputeServiceLiveTes
if (o == null || !this.getClass().equals(o.getClass()))
return false;
PayloadEquals other = (PayloadEquals) o;
- return this.expected == null && other.expected == null || this.expected != null
- && this.expected.equals(other.expected);
+ return (this.expected == null && other.expected == null) ||
+ (this.expected != null && this.expected.equals(other.expected));
}
@Override
http://git-wip-us.apache.org/repos/asf/jclouds/blob/25baad6d/providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/compute/strategy/CreateKeyPairPlacementAndSecurityGroupsAsNeededAndReturnRunOptions.java
----------------------------------------------------------------------
diff --git a/providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/compute/strategy/CreateKeyPairPlacementAndSecurityGroupsAsNeededAndReturnRunOptions.java b/providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/compute/strategy/CreateKeyPairPlacementAndSecurityGroupsAsNeededAndReturnRunOptions.java
index 371271c..782e376 100644
--- a/providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/compute/strategy/CreateKeyPairPlacementAndSecurityGroupsAsNeededAndReturnRunOptions.java
+++ b/providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/compute/strategy/CreateKeyPairPlacementAndSecurityGroupsAsNeededAndReturnRunOptions.java
@@ -179,8 +179,8 @@ public class CreateKeyPairPlacementAndSecurityGroupsAsNeededAndReturnRunOptions
@Override
protected boolean userSpecifiedTheirOwnGroups(TemplateOptions options) {
- return options instanceof AWSEC2TemplateOptions
- && !AWSEC2TemplateOptions.class.cast(options).getGroupIds().isEmpty()
+ return (options instanceof AWSEC2TemplateOptions
+ && !AWSEC2TemplateOptions.class.cast(options).getGroupIds().isEmpty())
|| super.userSpecifiedTheirOwnGroups(options);
}
http://git-wip-us.apache.org/repos/asf/jclouds/blob/25baad6d/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java
----------------------------------------------------------------------
diff --git a/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java b/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java
index 912f554..99ec4e7 100644
--- a/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java
+++ b/providers/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/compute/GoogleComputeEngineServiceAdapter.java
@@ -183,7 +183,7 @@ public final class GoogleComputeEngineServiceAdapter
// Add lookup for InstanceToNodeMetadata
diskURIToImage.getUnchecked(instance.get().disks().get(0).source());
- if (options.autoCreateWindowsPassword() != null && options.autoCreateWindowsPassword()
+ if ((options.autoCreateWindowsPassword() != null && options.autoCreateWindowsPassword())
|| OsFamily.WINDOWS == template.getImage().getOperatingSystem().getFamily()) {
Map<String, ?> params = ImmutableMap.of("instance", instance, "zone", zone, "email", create.user(), "userName", credentials.getUser());
String password = windowsPasswordGenerator.apply(params);