> @@ -83,7 +84,8 @@ public ContentMetadataBuilder expires(@Nullable Date 
> expires) {
>     }
>  
>     public ContentMetadata build() {
> -      return new BaseImmutableContentMetadata(contentType, contentLength, 
> contentMD5, contentDisposition,
> +      return new BaseImmutableContentMetadata(contentType, contentLength,
> +               contentMD5 == null ? null : contentMD5.asBytes(), 
> contentDisposition,

How about changing `BaseImmutableContentMetadata` to accept a HashCode 
constructor arg rather than a `byte[]`? Then we can save ourselves the `null` 
handling here?

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

Reply via email to