[libvirt] [PATCHv2 02/13] snapshot: use new virsh function for snapshot-list

[Peter Krempa]

On 06/15/12 06:18, Eric Blake wrote:
> Operating on a list of snapshot objects looks so much simpler.
> In particular, since the helper function already trimmed out
> irrelevant entries, we no longer have quite so many special cases
> on finding the first snapshot to operate on.  Also, vshTreePrint
> no longer has a generic callback struct; both clients now pass
> something different according to their own needs.
>
> * tools/virsh.c (cmdSnapshotList): Use previous patches.
> (vshTreeArrayLookup): Rename...
> (vshNodeListLookup): ...now that it only has one client.
> (cmdNodeListDevices): Adjust caller.
> ---
>
> v2: rename vshNodeListLookup
>
>   tools/virsh.c |  225 +++++++++++----------------------------------------------
>   1 file changed, 41 insertions(+), 184 deletions(-)
>

ACK; the listing functions look much better after splitting out getter code.

Peter