> @@ -171,7 +170,7 @@ public boolean removeSecurityGroup(String id) {
>
> ListOptions options = new ListOptions.Builder().filter("network eq
> .*/" + id);
>
> - FluentIterable<Firewall> fws =
> api.getFirewallApiForProject(userProject.get()).list(options).concat();
> + FluentIterable<Firewall> fws =
> api.getFirewallApiForProject(userProject.get()).list(options).toPagedIterable().concat();
Do you mean this?
```java
FluentIterable.from(api.getFirewallApiForProject(userProject.get())
.list(options)
.toPagedIterable()
.concat());
```
The
[PagedIterable#concat](https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/collect/PagedIterable.java#L73)
already returns a FluentIterable, so that won't be needed (and that method in
Guava is deprecated). Are you in the direction of changing the
`PagedIterable.concat()` method to have a regular `Iterable` in the signature?
Or am I missing something?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/62/files#r19263534
