> +                                                    
> GroupNamingConvention.Factory namingConvention,
> +                                                    
> LoadingCache<NetworkAndAddressRange, Network> networkCreator,
> +                                                    Function<Network, 
> SecurityGroup> groupConverter,
> +                                                    @Named("global") 
> Predicate<AtomicReference<Operation>> operationDonePredicate,
> +                                                    
> @Named(OPERATION_COMPLETE_INTERVAL) Long operationCompleteCheckInterval,
> +                                                    
> @Named(OPERATION_COMPLETE_TIMEOUT) Long operationCompleteCheckTimeout) {
> +      this.api = checkNotNull(api, "api");
> +      this.userProject = checkNotNull(userProject, "userProject");
> +      this.namingConvention = checkNotNull(namingConvention, 
> "namingConvention");
> +      this.networkCreator = checkNotNull(networkCreator, "networkCreator");
> +      this.groupConverter = checkNotNull(groupConverter, "groupConverter");
> +      this.operationCompleteCheckInterval = 
> checkNotNull(operationCompleteCheckInterval,
> +              "operation completed check interval");
> +      this.operationCompleteCheckTimeout = 
> checkNotNull(operationCompleteCheckTimeout,
> +              "operation completed check timeout");
> +      this.operationDonePredicate = operationDonePredicate;

`checkNotNull` here too?

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

Reply via email to