> @@ -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
