> From: Paolo Bonzini [mailto:pbonz...@redhat.com]
> On 12/01/2015 13:01, Pavel Dovgalyuk wrote:
> > +    default:
> > +    case QEMU_CLOCK_VIRTUAL:
> > +        if ((replay_mode != REPLAY_MODE_NONE && !runstate_is_running())
> > +            || !replay_checkpoint(run_all ? 2 : 3)) {
> > +            return false;
> > +        }
> > +        break;
> 
> Please document the meaning of the numbers by making an enum.  

The numbers have no meaning. They just have to be distinct in different places.

> Why do you have to distinguish run_all?

It seems to be early versions artifact. I'll remove it.

Pavel Dovgalyuk


Reply via email to