On Fri, Jul 30, 2010 at 10:39 AM, Luiz Capitulino
<lcapitul...@redhat.com> wrote:
>>
>> TODO: I have no clue on how to create a timestamp string when using Windows.
>
> So, what happens on windows?
>

I've found some code in bdrv_snapshot_dump() that formats a timestamp
when using Windows. I will use it as a basis.

> Also, please, avoid making changes which are unrelated to the patch, like:
>
>>   the_end:
>> -    if (saved_vm_running)
>> +    if (saved_vm_running) {
>>          vm_start();
>> +    }
>>  }
>>

Ack. Coding style cleanups must have a separate patch :-D

Regards,

Miguel

Reply via email to