it was a result of trying to affect the least amount of variable names
(they were originally method local and named api).  I'll change all of
them.  np.


On Tue, Apr 9, 2013 at 9:43 AM, Everett Toews <notifications@github.com>wrote:

> In
> apis/openstack-nova/src/test/java/org/jclouds/openstack/nova/v2_0/extensions/KeyPairApiLiveTest.java:
>
> > @@ -35,8 +35,8 @@
> >  public class KeyPairApiLiveTest extends BaseNovaApiLiveTest {
> >
> >     public void testListKeyPairs() throws Exception {
> > -      for (String zoneId : novaContext.getApi().getConfiguredZones()) {
> > -         KeyPairApi api = novaContext.getApi().getKeyPairExtensionForZone(zoneId).get();
> > +      for (String zoneId : api.getConfiguredZones()) {
> > +         KeyPairApi api = this.api.getKeyPairExtensionForZone(zoneId).get();
>
> Why use api in the foreach and this.api in the line below?
>
> Same comment for below and wherever else this might be done.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jclouds/jclouds/pull/1508/files#r3717362>
> .
>