On 04/21/2010 10:01 AM, Chris Lalancette wrote: > Signed-off-by: Chris Lalancette <clala...@redhat.com> > --- > tools/Makefile.am | 1 + > tools/virsh.c | 55 > +++++++++++++++++++++++++++++++++++++++++++++++++++++
Nothing in tools/virsh.pod? Hmm, we ought to modify docs/api_extension to mention documenting new virsh commands (step 8/8). > + if (virDomainQemuMonitorCommand(dom, monitor_cmd, &result, 0) < 0) > + goto cleanup; > + > + fprintf(stdout, "%s\n", result); Isn't puts(result) more efficient, for the same end result? To help keep docs in sync, I'll withhold an ack until seeing the .pod updates in the respin :) -- Eric Blake ebl...@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list