> + @Override
> + public Iterable<VirtualMachine> listNodes() {
> + Closer closer = Closer.create();
> + VSphereServiceInstance instance = serviceInstance.get();
> + closer.register(instance);
> +
> + try {
> + try {
> + return listNodes(instance);
> + } catch (Throwable e) {
> + logger.error("Can't find vm", e);
> + throw closer.rethrow(e);
> + } finally {
> + closer.close();
> + }
> + } catch (Throwable t) {
Is this catch needed? Aren't all exceptions already caught?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/61/files#r18061519
