Hi Michael,

[cut]
> Exception messages shall never end with punctuation.
[cut]
> One empty line between class-level methods/functions.
[cut]
> You need to reindent the last line, too.

Thanks for the formatting review, I will apply those changes immediately.

> Now, isn't there a race condition between starting KVM and setting the 
> password?

I think there is no race condition, because utils.RunCmd() blocks until
the KVM executable was launched, at which point (if it didn't fail) I
suppose that the monitor will already be listening -- unless KVM runs that
asynchronously.

Thanks,
Andrea

Reply via email to