Quoting Stéphane Graber (stgra...@ubuntu.com):
> > @@ -134,9 +133,12 @@ int lxc_execute(const char *name, char *const argv[], 
> > int quiet,
> >             .argv = argv,
> >             .quiet = quiet
> >     };
> > +   int ret;
> >  
> >     if (lxc_check_inherited(conf, -1))
> >             return -1;
> >  
> > -   return __lxc_start(name, conf, &execute_start_ops, &args);
> > +   ret = __lxc_start(name, conf, &execute_start_ops, &args);
> > +
> > +   return ret;
> >  }
> 
> What's the reason for that bit? Looks to me as functionally identical.

D'oh - good catch.  The reason for it was that at first I had the
rename_phys_on_shutdown in the lxc_start callers, not in __lxc_start
itself.  I moved it right at the end, but forgot about this now-
unneeded bit.

thanks,
-serge

------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
Lxc-devel mailing list
Lxc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-devel

Reply via email to