Changeset 14224

Timestamp:

11/14/08 17:17:20 (2 months ago)

Author:

vboxsync

Message:

Main: Added ISystemProperties::defaultHardDiskFormat.

Files:

• trunk/src/VBox/Main/HardDisk2Impl.cpp (modified) (1 diff)
• trunk/src/VBox/Main/SystemPropertiesImpl.cpp (modified) (6 diffs)
• trunk/src/VBox/Main/VirtualBoxImpl.cpp (modified) (1 diff)
• trunk/src/VBox/Main/idl/VirtualBox.xidl (modified) (4 diffs)
• trunk/src/VBox/Main/include/SystemPropertiesImpl.h (modified) (4 diffs)
• trunk/src/VBox/Main/xml/VirtualBox-settings-common.xsd (modified) (1 diff)

trunk/src/VBox/Main/HardDisk2Impl.cpp

@@ -472 +472 @@
-                         const BSTR aLocation)
-{
-
-
+
+
+
-
-    /* Enclose the state transition NotReady->InInit->Ready */

trunk/src/VBox/Main/SystemPropertiesImpl.cpp

@@ -75 +75 @@
-    setDefaultMachineFolder (NULL);
-    setDefaultHardDiskFolder (NULL);
+    setDefaultHardDiskFormat (NULL);
+
-    setRemoteDisplayAuthLibrary (NULL);
-
@@ -362 +364 @@
-}
-
+STDMETHODIMP SystemProperties::COMGETTER(DefaultHardDiskFormat) (BSTR *aDefaultHardDiskFormat)
+{
+    if (!aDefaultHardDiskFormat)
+        return E_POINTER;
+
+    AutoCaller autoCaller (this);
+    CheckComRCReturnRC (autoCaller.rc());
+
+    AutoReadLock alock (this);
+
+    mDefaultHardDiskFormat.cloneTo (aDefaultHardDiskFormat);
+
+    return S_OK;
+}
+
+STDMETHODIMP SystemProperties::COMSETTER(DefaultHardDiskFormat) (INPTR BSTR aDefaultHardDiskFormat)
+{
+    AutoCaller autoCaller (this);
+    CheckComRCReturnRC (autoCaller.rc());
+
+    /* VirtualBox::saveSettings() needs a write lock */
+    AutoMultiWriteLock2 alock (mParent, this);
+
+    HRESULT rc = setDefaultHardDiskFormat (aDefaultHardDiskFormat);
+    if (SUCCEEDED (rc))
+        rc = mParent->saveSettings();
+
+    return rc;
+}
+
-STDMETHODIMP SystemProperties::COMGETTER(RemoteDisplayAuthLibrary) (BSTR *aRemoteDisplayAuthLibrary)
-{
@@ -508 +540 @@
-    bstr = properties.stringValue ("defaultHardDiskFolder");
-    rc = setDefaultHardDiskFolder (bstr);
+    CheckComRCReturnRC (rc);
+
+    bstr = properties.stringValue ("defaultHardDiskFormat");
+    rc = setDefaultHardDiskFormat (bstr);
-    CheckComRCReturnRC (rc);
-
@@ -549 +585 @@
-    if (mDefaultHardDiskFolder)
-        properties.setValue <Bstr> ("defaultHardDiskFolder", mDefaultHardDiskFolder);
+
+    if (mDefaultHardDiskFormat)
+        properties.setValue <Bstr> ("defaultHardDiskFormat", mDefaultHardDiskFormat);
-
-    if (mRemoteDisplayAuthLibrary)
@@ -610 +649 @@
-}
-
+HRESULT SystemProperties::setDefaultHardDiskFormat (const BSTR aFormat)
+{
+    if (aFormat && *aFormat)
+        mDefaultHardDiskFormat = aFormat;
+    else
+        mDefaultHardDiskFormat = "VDI";
+
+    return S_OK;
+}
+
-HRESULT SystemProperties::setRemoteDisplayAuthLibrary (const BSTR aPath)
-{
-    Utf8Str path;
-    if (aPath && *aPath)
-path
+mRemoteDisplayAuthLibrary
-    else
-
-
-
+
-
-    return S_OK;
@@ -625 +671 @@
-HRESULT SystemProperties::setWebServiceAuthLibrary (const BSTR aPath)
-{
-    Utf8Str path;
-    if (aPath && *aPath)
-path
+mWebServiceAuthLibrary
-    else
-
-
-
-
-
-
+
+
+
+

trunk/src/VBox/Main/VirtualBoxImpl.cpp

@@ -1060 +1060 @@
-    /* we don't access non-const data members so no need to lock */
-
+    Bstr format = aFormat;
+    if (format.isEmpty())
+    {
+        AutoReadLock propsLock (systemProperties());
+        format = systemProperties()->defaultHardDiskFormat();
+    }
+
-    HRESULT rc = E_FAIL;
-
-    ComObjPtr <HardDisk2> hardDisk;
-    hardDisk.createObject();
-aF
+f
-
-    if (SUCCEEDED (rc))

trunk/src/VBox/Main/idl/VirtualBox.xidl

@@ -1386 +1386 @@
-        the <link to="#hardDisks2"/> array. See IHardDisk2 for more details.
-
-
-
-
+
+
+
+
+
+
-
-        Note that the format of the location string is storage format specific.
@@ -1396 +1399 @@
-      <param name="format" type="wstring" dir="in">
-        <desc>
-S
+Idetifier of the 
-        </desc>
-      </param>
@@ -5269 +5272 @@
-     name="ISystemProperties"
-     extends="$unknown"
-c7c060a9-91af-4e74-9188-a6b15a725e49
+604afeba-5963-4d12-a577-902ffb96352a
-     wsmap="managed"
-     >
@@ -5415 +5418 @@
-        <see>
-          <link to="IHardDiskFormat"/>,
+        </see>
+      </desc>
+    </attribute>
+
+    <attribute name="defaultHardDiskFormat" type="wstring">
+      <desc>
+        Identifier of the default hard disk format used by VirtualBox.
+
+        The hard disk format set by this attribute is used by VirtualBox
+        when the hard disk format was not specified explicitly. One example is
+        <link to="IVirtualBox::createHardDisk2()"/> with the <tt>null</tt>
+        format argument. A more complex example is implicit creation of
+        differencing hard disks when taking a snapshot of a virtual machine:
+        this operation will try to use a format of the parent hard disk first
+        and if this format does not support differencing hard disks the default
+        format specified by this argument will be used.
+
+        The list of supported hard disk formats may be obtained by the
+        <link  to="#defaultHardDiskFormats"/> call. Note that the default
+        hard disk format must have a capability to create differencing hard
+        disks; otherwise opeartions that create hard disks implicitly may fail
+        unexpectedly.
+
+        The initial value of this property is <tt>VDI</tt> in the current
+        version of the VirtualBox product, but may change in the future.
+
+        <note>
+          Setting this property to <tt>null</tt> will restore the
+          initial value.
+        </note>
+
+        <see>
+          <link to="#hardDiskFormats"/>,
+          <link to="IHardDiskFormat:id"/>,
+          <link to="IVirtualBox::createHardDisk2()"/>
-        </see>
-      </desc>

trunk/src/VBox/Main/include/SystemPropertiesImpl.h

@@ -80 +80 @@
-    STDMETHOD(COMSETTER(DefaultHardDiskFolder)) (INPTR BSTR aDefaultHardDiskFolder);
-    STDMETHOD(COMGETTER(HardDiskFormats)) (ComSafeArrayOut (IHardDiskFormat *, aHardDiskFormats));
+    STDMETHOD(COMGETTER(DefaultHardDiskFormat)) (BSTR *aDefaultHardDiskFolder);
+    STDMETHOD(COMSETTER(DefaultHardDiskFormat)) (INPTR BSTR aDefaultHardDiskFolder);
-    STDMETHOD(COMGETTER(RemoteDisplayAuthLibrary)) (BSTR *aRemoteDisplayAuthLibrary);
-    STDMETHOD(COMSETTER(RemoteDisplayAuthLibrary)) (INPTR BSTR aRemoteDisplayAuthLibrary);
@@ -103 +105 @@
-    const Bstr &defaultHardDiskFolderFull() const { return mDefaultHardDiskFolderFull; }
-
+    /** Default hard disk format. Not thread safe (use object lock). */
+    const Bstr &defaultHardDiskFormat() const { return mDefaultHardDiskFormat; }
+
-    // for VirtualBoxSupportErrorInfoImpl
-    static const wchar_t *getComponentName() { return L"SystemProperties"; }
@@ -112 +117 @@
-    HRESULT setDefaultMachineFolder (const BSTR aPath);
-    HRESULT setDefaultHardDiskFolder (const BSTR aPath);
+    HRESULT setDefaultHardDiskFormat (const BSTR aFormat);
+
-    HRESULT setRemoteDisplayAuthLibrary (const BSTR aPath);
-    HRESULT setWebServiceAuthLibrary (const BSTR aPath);
@@ -121 +128 @@
-    Bstr mDefaultHardDiskFolder;
-    Bstr mDefaultHardDiskFolderFull;
+    Bstr mDefaultHardDiskFormat;
-
-    HardDiskFormatList mHardDiskFormats;

trunk/src/VBox/Main/xml/VirtualBox-settings-common.xsd

@@ -305 +305 @@
-  <xsd:attribute name="defaultMachineFolder" type="TLocalFile"/>
-  <xsd:attribute name="defaultHardDiskFolder" type="TLocalFile"/>
+  <xsd:attribute name="defaultHardDiskFormat" type="TNonEmptyString"/>
-  <xsd:attribute name="remoteDisplayAuthLibrary" type="TLocalFile"/>
-  <xsd:attribute name="webServiceAuthLibrary" type="TLocalFile"/>