Strictly speaking this is not needed right now. However, next
commits will require dispose function to exist.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
---
 src/rpc/virnetsaslcontext.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/rpc/virnetsaslcontext.c b/src/rpc/virnetsaslcontext.c
index a7b891feb6..f1f8bdc855 100644
--- a/src/rpc/virnetsaslcontext.c
+++ b/src/rpc/virnetsaslcontext.c
@@ -52,6 +52,7 @@ struct _virNetSASLSession {
 
 static virClassPtr virNetSASLContextClass;
 static virClassPtr virNetSASLSessionClass;
+static void virNetSASLContextDispose(void *obj);
 static void virNetSASLSessionDispose(void *obj);
 
 static int virNetSASLContextOnceInit(void)
@@ -59,7 +60,7 @@ static int virNetSASLContextOnceInit(void)
     if (!(virNetSASLContextClass = virClassNew(virClassForObjectLockable(),
                                                "virNetSASLContext",
                                                sizeof(virNetSASLContext),
-                                               NULL)))
+                                               virNetSASLContextDispose)))
         return -1;
 
     if (!(virNetSASLSessionClass = virClassNew(virClassForObjectLockable(),
@@ -686,6 +687,11 @@ ssize_t virNetSASLSessionDecode(virNetSASLSessionPtr sasl,
     return ret;
 }
 
+void virNetSASLContextDispose(void *obj ATTRIBUTE_UNUSED)
+{
+    /* nada */
+}
+
 void virNetSASLSessionDispose(void *obj)
 {
     virNetSASLSessionPtr sasl = obj;
-- 
2.16.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

On Fri, Apr 13, 2018 at 04:47:13PM +0200, Michal Privoznik wrote:
> Strictly speaking this is not needed right now. However, next

The first sentence only makes sense in the context of the patch series review,
that will not be the case for someone looking at it in the future - plain
"Future commits rely on the presence of this callback" should suffice.

Reviewed-by: Erik Skultety <eskultet@redhat.com>

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

On Fri, Apr 13, 2018 at 04:47:13PM +0200, Michal Privoznik wrote:
> Strictly speaking this is not needed right now. However, next
> commits will require dispose function to exist.
>
> Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
> ---

...

>  src/rpc/virnetsaslcontext.c | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> +void virNetSASLContextDispose(void *obj ATTRIBUTE_UNUSED)
> +{
> +    /* nada */

One more thing, either leave the body empty, or replace the commentary with a
plain "return" call.

Erik

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list