> +
> +   public abstract int modifiedIndex();
> +
> +   public abstract int ttl();
> +
> +   @Nullable
> +   public abstract String value();
> +
> +   Node() {
> +   }
> +
> +   @SerializedNames({ "createdIndex", "dir", "nodes", "expiration", "key", 
> "modifiedIndex", "ttl", "value" })
> +   public static Node create(int createdIndex, boolean dir, List<Node> 
> nodes, String expiration, String key,
> +         int modifiedIndex, int ttl, String value) {
> +      return new AutoValue_Node(createdIndex, dir,
> +            ImmutableList.copyOf(nodes != null ? nodes : 
> ImmutableList.<Node> of()), expiration, key, modifiedIndex,

Why copying an empty list? Why not just: `nodes != null ? 
ImmutableList.copyOf(nodes) : ImmutableList.<Node> of()`

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/234/files#r53013341

Reply via email to