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