You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@deltacloud.apache.org by David Lutterkort <lu...@redhat.com> on 2012/05/10 02:20:12 UTC

Re: [PATCH core 06/32] Core: Added support for autoloading all Deltacloud models

On Tue, 2012-04-17 at 15:39 +0200, mfojtik@redhat.com wrote:
> From: Michal Fojtik <mf...@redhat.com>

This also is a lot of magic to me - is there really that much to gain
from not having to add a new model to the list of requires here ?

David



Re: [PATCH core 06/32] Core: Added support for autoloading all Deltacloud models

Posted by David Lutterkort <lu...@redhat.com>.
On Thu, 2012-05-10 at 10:06 +0200, Michal Fojtik wrote:
> On 05/10/12, David Lutterkort wrote:
> > On Tue, 2012-04-17 at 15:39 +0200, mfojtik@redhat.com wrote:
> > > From: Michal Fojtik <mf...@redhat.com>
> > 
> > This also is a lot of magic to me - is there really that much to gain
> > from not having to add a new model to the list of requires here ?
> 
> Yeah, well I usually end up with code like this when I need to type
> the same thing too many times. But not a problem, I'll add the require
> there to make it more clean.

Think of the poor guy who looks at this for the first time ;)

David



Re: [PATCH core 06/32] Core: Added support for autoloading all Deltacloud models

Posted by Michal Fojtik <mf...@redhat.com>.
On 05/10/12, David Lutterkort wrote:
> On Tue, 2012-04-17 at 15:39 +0200, mfojtik@redhat.com wrote:
> > From: Michal Fojtik <mf...@redhat.com>
> 
> This also is a lot of magic to me - is there really that much to gain
> from not having to add a new model to the list of requires here ?

Yeah, well I usually end up with code like this when I need to type
the same thing too many times. But not a problem, I'll add the require
there to make it more clean.

  -- michal

-- 
Michal Fojtik
Sr. Software Engineer, Deltacloud API (http://deltacloud.org)