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

Re: [libvirt] [PATCH 00/50] Atomic APIs to list objects



On 07/19/2012 10:40 AM, Osier Yang wrote:
> Except the already supported APIs for domain and domain snapshot,
> this series add the APIs for all the left objects, including
> storage pool, storage vol, network, interface, node device,
> nwfilter, secret.

Wow, you've been busy.

> By the way, I'm wondering if should split the virsh wood, it's really
> big now, and still growing up. It might take a bit more time to
> compile if it's splitted into multiple files (split it by the command
> group?) though, but it's good for maintaining and eyes (I have a small
> screen). Any opinions?

Yes, it's time to split.  One .c file per command group is a reasonable
start (all domain commands in one file, all interface commands in one
file, and so forth, with virsh.c providing main() and command parsing
utilities common to the use of all the commands).

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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