You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jclouds.apache.org by Andrew Phillips <no...@github.com> on 2013/07/01 15:54:32 UTC

Re: [jclouds-labs] JCLOUDS-156: Cleanup Abiquo provider (#10)

> @@ -91,7 +93,15 @@ public void save() {
>        target = context.getApi().getInfrastructureApi().createMachine(rack.unwrap(), target);
>     }
>  
> -   @Override
> +   public void delete() {
> +      context.getApi().getInfrastructureApi().deleteMachine(target);
> +      target = null;
> +   }

Perhaps a stupid question (don't know the code)...can "bad stuff" (NPEs etc.) happen if the user later calls e.g. `update` with a `null` target? If so, should there be some documentation for this method along the lines of "once you call this, don't expect X, Y or Z to work"...

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