[libvirt] [PATCH] docs: silence warnings about generated API docs

Eric Blake eblake at redhat.com
Wed Mar 9 03:54:35 UTC 2011


I noticed these while testing 'make dist'.

Parsing ./../src/util/event.c
Function comment for virEventRegisterDefaultImpl lacks description of return value
Function comment for virEventRunDefaultImpl lacks description of return value
Parsing ./../src/util/virterror.c
Missing comment for function virSetErrorLogPriorityFunc

* src/util/event.c (virEventRegisterDefaultImpl)
(virEventRunDefaultImpl): Document return types.
* src/util/virterror.c (virSetErrorLogPriorityFunc): Provide docs.
---

Enough material that I thought I'd wait for an ack.

 src/util/event.c     |    6 +++++-
 src/util/virterror.c |    8 +++++++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/util/event.c b/src/util/event.c
index 0d88b55..bc0129c 100644
--- a/src/util/event.c
+++ b/src/util/event.c
@@ -1,8 +1,8 @@
 /*
  * event.c: event loop for monitoring file handles
  *
+ * Copyright (C) 2007, 2011 Red Hat, Inc.
  * Copyright (C) 2007 Daniel P. Berrange
- * Copyright (C) 2007 Red Hat, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -140,6 +140,8 @@ void virEventRegisterImpl(virEventAddHandleFunc addHandle,
  * Once registered, the application can invoke
  * virEventRunDefaultImpl in a loop to process
  * events
+ *
+ * Returns 0 on success, -1 on failure.
  */
 int virEventRegisterDefaultImpl(void)
 {
@@ -178,6 +180,8 @@ int virEventRegisterDefaultImpl(void)
  *    if (virEventRunDefaultImpl() < 0)
  *       ...print error...
  *  }
+ *
+ * Returns 0 on success, -1 on failure.
  */
 int virEventRunDefaultImpl(void)
 {
diff --git a/src/util/virterror.c b/src/util/virterror.c
index aaa3720..f136054 100644
--- a/src/util/virterror.c
+++ b/src/util/virterror.c
@@ -1,7 +1,7 @@
 /*
  * virterror.c: implements error handling and reporting code for libvirt
  *
- * Copy:  Copyright (C) 2006, 2008-2010 Red Hat, Inc.
+ * Copy:  Copyright (C) 2006, 2008-2011 Red Hat, Inc.
  *
  * See COPYING.LIB for the License of this software
  *
@@ -1344,6 +1344,12 @@ void virReportOOMErrorFull(int domcode,
                       virerr, NULL, NULL, -1, -1, virerr, NULL);
 }

+/**
+ * virSetErrorLogPriorityFunc:
+ * @func: function to install
+ *
+ * Install a function used to filter error logging based on error priority.
+ */
 void virSetErrorLogPriorityFunc(virErrorLogPriorityFunc func)
 {
     virErrorLogPriorityFilter = func;
-- 
1.7.4




More information about the libvir-list mailing list