Jennifer Chou wrote on 03/23/10 08:00 AM:
> Not sure if you need to add defaultValue to Param, but I saw them added
> in other places.
>
> LocalDomainCommand.java - looks ok
>
> DeleteDomainCommand.java - looks ok to me.
>
> StopDomainCommand.java - look ok
>
> StartDomainCommand.java: add defaultValue="false"?
>
> 70
> 71 @Param(optional = true,* defaultValue="false"*)
> 72 private boolean verbose;
> 73
> 74 @Param(optional = true,**defaultValue="false"**)
> 75 private boolean upgrade;
> 76
> 77 @Param(optional = true,**defaultValue="false"**)
> 78 private boolean debug;
>
>
> MonitorCommand.java - add defaultValue="30"?
>
> 61 @Param(optional = true,*defaultValue="30"*)
> 62 private int interval = 30; // default 30 seconds
Yup, I should fix these too.