nacx commented on this pull request.


> +            @ConstructorProperties({ "devices", "meta" })
+            public Devices(List<Device> items, Meta meta) {
+                super(items, meta);
+            }
+        }
+
+        private static class ToPagedIterable extends 
BaseToPagedIterable<Device, ListOptions> {
+
+            @Inject ToPagedIterable(PacketApi api, Function<Href, ListOptions> 
linkToOptions) {
+                super(api, linkToOptions);
+            }
+
+           @Override
+           protected List<Object> getArgs(GeneratedHttpRequest request) {
+              return request.getCaller().get().getArgs();
+           }

Oh, I see. It is a nested API that needs the args from the caller, not from 
their own methods. In that case, this function should better extend 
[Arg0ToPagedIterable.FromCaller](https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/collect/internal/Arg0ToPagedIterable.java#L53),
 which is intended for nested APIs.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/345

Reply via email to