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

Re: [libvirt] [PATCHv2 01/16] command: move all docs into .c file



On Tue, Jul 19, 2011 at 10:20:24PM -0600, Eric Blake wrote:
> We already have a precedent of function documentation in C files,
> where it is closer to the implementation (witness libvirt.h vs.
> libvirt.c); maintaining docs in both files risks docs going stale.
> 
> While I was at it, I used consistent doxygen style on all comments.
> 
> * src/util/command.h: Remove duplicate docs, and move unique
> documentation...
> * src/util/command.c: ...here.
> Suggested by Matthias Bolte.
> ---
> 
> prereq patch to 2/16 (not directly related to bypass-cache, but I
> don't want to rebase across this one)
> 
> v2: no change from v1, previously posted here:
> https://www.redhat.com/archives/libvir-list/2011-July/msg00879.html
> 
>  src/util/command.c |  371 ++++++++++++++++++++++++++++++++++++++++------------
>  src/util/command.h |  223 ++------------------------------
>  2 files changed, 299 insertions(+), 295 deletions(-)

ACK, trivial NFC

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|


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