Re: [patch 3/9] Re-factor chunks of qemuds config handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 22, 2007 at 11:04:37AM +0000, Mark McLoughlin wrote:
> The easiest way to explain where I'm coming from with
> this patch is to look at the current qemudLoadConfigXML().
> 
> It's used in three places:
> 
>   1) Loading at startup - you want (a) and (b), but you
>      want to check whether the name matches the filename
>      before doing (b)
> 
>   2) Create() - you want (a) and (b)
> 
>   3) Define() - you want (a), (b) and (c)
> 
> So, mostly I'm just splitting the function into three
> and making the logic above much more obvious. It should
> also make some of the autostart code much more
> straightforward.

Hard to tell from the raw patch exactly how it fits together, but the
principles of the patch all sound good to me - centralizing alot of
the pointer munging into one place & generally tidying stuff up. 

BTW, your mailer alias for this list is wrong - its generating a bogus
reply to against

   libvirt hacker dudes <libvirt-list@xxxxxxxxxx>

Instead of

   libvirt hacker dudes <libvir-list@xxxxxxxxxx>

Note, the extra 't' in the former - which makes replying to your mails a
little error prone - several went to dev null...

Dan
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]