Re: [PATCH 3/6] vbox: Add glue layer for MSCOM on Windows

[Matthias Bolte <matthias.bolte@xxxxxxxxxxxxxx>]

2010/12/17 Eric Blake <eblake@xxxxxxxxxx>:
> On 12/17/2010 11:56 AM, Matthias Bolte wrote:
>> Don't require dlopen, but link to ole32 and oleaut32 on Windows.
>>
>> Don't expose g_pVBoxFuncs anymore. It was only used to get the
>> version of the API. Make VBoxCGlueInit return the version instead.
>> This simplifies the implementation of the MSCOM glue layer.
>>
>> Get the VirtualBox version from the registry.
>>
>> Add a dummy implementation of the nsIEventQueue to the MSCOM glue
>> as there seems to be no direct equivalent with MSCOM. It might be
>> implemented using the normal window message loop. This requires
>> additional investigation.
>> ---
>> Âconfigure.ac        |  15 +-
>> Âpo/POTFILES.in       |  Â1 +
>> Âsrc/Makefile.am      Â|  Â9 +-
>> Âsrc/vbox/vbox_MSCOMGlue.c Â| Â649 ++++++++++++++++++++++++++++++++++++++++++++
>> Âsrc/vbox/vbox_MSCOMGlue.h Â| Â 33 +++
>> Âsrc/vbox/vbox_XPCOMCGlue.c | Â 46 ++--
>> Âsrc/vbox/vbox_XPCOMCGlue.h | Â Â4 +-
>> Âsrc/vbox/vbox_driver.c   |  Â6 +-
>> Âsrc/vbox/vbox_glue.c    |  29 ++
>> Âsrc/vbox/vbox_glue.h    |  32 +++
>> Âsrc/vbox/vbox_tmpl.c    |  12 +-
>> Â11 files changed, 798 insertions(+), 38 deletions(-)
>> Âcreate mode 100644 src/vbox/vbox_MSCOMGlue.c
>> Âcreate mode 100644 src/vbox/vbox_MSCOMGlue.h
>> Âcreate mode 100644 src/vbox/vbox_glue.c
>> Âcreate mode 100644 src/vbox/vbox_glue.h
>>
>> +
>> + Â Âcase "$host" in
>> + Â Â Â*-*-mingw* | *-*-msvc*) MSCOM_LIBS="-lole32 -loleaut32" ;;
>> + Â Â Â*) MSCOM_LIBS= ;;
>> + Â Âesac
>> + Â ÂAC_SUBST([MSCOM_LIBS])
>
> Should we also modify the tail end of ./configure output to display what
> values were set in $MSCOM_LIBS, as is done for various other libraries?
> ÂBut that's a minor nit.

Yes, good idea. I'll fold this in.

@@ -2430,6 +2430,11 @@ AC_MSG_NOTICE([      nl: $LIBNL_CFLAGS $LIBNL_LIBS])
 else
 AC_MSG_NOTICE([      nl: no])
 fi
+if test "$with_vbox" = "yes" && test -n "$MSCOM_LIBS" ; then
+AC_MSG_NOTICE([   mscom: $MSCOM_LIBS])
+else
+AC_MSG_NOTICE([   mscom: no])
+fi
 AC_MSG_NOTICE([])
 AC_MSG_NOTICE([Test suite])
 AC_MSG_NOTICE([])

>> +
>> +PFNVBOXGETXPCOMCFUNCTIONS g_pfnGetFunctions = NULL;
>> +
>> +static unsigned long vboxVersion;
>> +static IVirtualBox *vboxVirtualBox = NULL;
>> +static ISession *vboxSession = NULL;
>
> Is the explicit NULL initialization needed here, or are you okay relying
> on C-mandated 0-initialization of .bss variables?

Sure, no problem. I'll remove the NULL initialization.

> At any rate, ACK.
>

Thanks, pushed.

Matthias

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