jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Phillips <notificati...@github.com>
Subject Re: [jclouds-labs-google] JCLOUDS-458:Added Blobstore Abstraction (#48)
Date Sat, 20 Sep 2014 03:24:26 GMT
> +import org.jclouds.blobstore.strategy.IfDirectoryReturnNameStrategy;
> +import org.jclouds.googlecloudstorage.domain.GCSObject;
> +import com.google.common.base.Function;
> +import com.google.common.collect.ImmutableMap;
> +
> +@Singleton
> +public class ObjectToBlobMetadata implements Function<GCSObject, MutableBlobMetadata>
{
> +   private final IfDirectoryReturnNameStrategy ifDirectoryReturnName;
> +
> +   @Inject
> +   public ObjectToBlobMetadata(IfDirectoryReturnNameStrategy ifDirectoryReturnName)
{
> +      this.ifDirectoryReturnName = ifDirectoryReturnName;
> +   }
> +
> +   public MutableBlobMetadata apply(GCSObject from) {
> +      if (from == null)

I'd ask for `{...}` here but I see other places where we don't use them in situations like
this.

Thoughts on that, @andrewgaul!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/48/files#r17817580
Mime
View raw message