> +
> +   @ConstructorProperties({
> +      "id", "creationTimestamp", "selfLink", "name", "description", 
> "validDiskSize",
> +      "deprecated", "zone", "defaultDiskSizeGb"
> +   })
> +   private DiskType(String id, Date creationTimestamp, URI selfLink, String 
> name, String description,
> +                  String validDiskSize, Deprecated deprecated, String zone,  
> long defaultDiskSizeGb){
> +      super(Kind.DISK_TYPE, id == null ? "" : id, creationTimestamp, 
> selfLink, name, description);
> +      this.validDiskSize = checkNotNull(validDiskSize, "validDiskSize of 
> %s", name);
> +      this.deprecated = fromNullable(deprecated);
> +      this.zone = checkNotNull(zone, "zone of %s", name);
> +      this.defaultDiskSizeGb = defaultDiskSizeGb;
> +   }
> +
> +   /**
> +    * @return An optional textual description of the valid disk size. For 
> example, "10GB-10TB."

The javadoc is based on the website. Should I change it to an Optional<String> 
or just add the @Nullable annotation somewhere? 

Im relatively new to Java.

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

Reply via email to