> +            result = Lists.newArrayList();
> +         }
> +         return hashList.iterator().next();
> +      }
> +
> +      /**
> +       * Builds the Hash and the TreeHash values of the payload.
> +       *
> +       * @return The calculated TreeHash.
> +       * @see <a 
> href="http://docs.aws.amazon.com/amazonglacier/latest/dev/checksum-calculations.html";
>  />
> +       */
> +      public static TreeHash buildTreeHashFromPayload(Payload payload) 
> throws IOException {
> +         InputStream is = null;
> +         try {
> +            is = checkNotNull(payload, "payload").openStream();
> +            ArrayList<HashCode> list = Lists.newArrayList();

`List<HashCode>`? Or are we really interested in this being an `ArrayList`? 
Alternatively, use an immutable list builder?

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

Reply via email to