Changeset 50899 in vbox for trunk/src/VBox/Main/src-server

Timestamp:

Mar 26, 2014 6:08:27 PM (11 years ago)

Author:

vboxsync

Message:

remove changes that break up log and error strings, introduced in 92972

Location:

trunk/src/VBox/Main/src-server

Files:

• ApplianceImpl.cpp (modified) (1 diff)
• MachineImplCloneVM.cpp (modified) (6 diffs)
• SnapshotImpl.cpp (modified) (2 diffs)
• generic/NetIf-generic.cpp (modified) (2 diffs)

trunk/src/VBox/Main/src-server/ApplianceImpl.cpp

@@ -937 +937 @@
         }
     }
-
-    Utf8Str str;
-    str = "Setting up progress object: ulTotalMB = %d, cDisks = %d, => cOperations = %d,";
-    str +=  "ulTotalOperationsWeight = %d, m->ulWeightForXmlOperation = %d\n";
-    Log((str.c_str(), m->ulTotalDisksMB, m->cDisks, cOperations, ulTotalOperationsWeight, m->ulWeightForXmlOperation));
+    Log(("Setting up progress object: ulTotalMB = %d, cDisks = %d, => cOperations = %d, ulTotalOperationsWeight = %d, m->ulWeightForXmlOperation = %d\n",
+         m->ulTotalDisksMB, m->cDisks, cOperations, ulTotalOperationsWeight, m->ulWeightForXmlOperation));
 
     rc = pProgress->init(mVirtualBox, static_cast<IAppliance*>(this),

trunk/src/VBox/Main/src-server/MachineImplCloneVM.cpp

@@ -677 +677 @@
                         pParent->i_getPreferredDiffFormat(),
                         Utf8StrFmt("%s%c", strSnapshotFolder.c_str(), RTPATH_DELIMITER),
-                        Guid::Empty /* empty media registry */);
+                                   Guid::Empty /* empty media registry */);
         if (FAILED(rc)) throw rc;
 
@@ -1037 +1037 @@
         /* The absolute name of the snapshot folder. */
         strTrgSnapshotFolder = Utf8StrFmt("%s%c%s", strTrgMachineFolder.c_str(), RTPATH_DELIMITER,
-                                                    trgMCF.machineUserData.strSnapshotFolder.c_str());
+                                          trgMCF.machineUserData.strSnapshotFolder.c_str());
 
         /* Should we rename the disk names. */
@@ -1168 +1168 @@
                             if (strSrcTest == strOldVMName)
                                 strNewName = Utf8StrFmt("%s%s", trgMCF.machineUserData.strName.c_str(),
-                                                                RTPathSuffix(Utf8Str(bstrSrcName).c_str()));
+                                                        RTPathSuffix(Utf8Str(bstrSrcName).c_str()));
                             else if (   strSrcTest.startsWith("{")
                                      && strSrcTest.endsWith("}"))
@@ -1177 +1177 @@
                                 if (temp_guid.isValid() && !temp_guid.isZero())
                                     strNewName = Utf8StrFmt("%s%s", newId.toStringCurly().c_str(),
-                                                                    RTPathSuffix(strNewName.c_str()));
+                                                            RTPathSuffix(strNewName.c_str()));
                             }
                             else
@@ -1350 +1350 @@
                 throw p->setError(VBOX_E_IPRT_ERROR,
                                   p->tr("Could not create snapshots folder '%s' (%Rrc)"),
-                                  strTrgSnapshotFolder.c_str(), vrc);
+                                        strTrgSnapshotFolder.c_str(), vrc);
         }
         /* Clone all save state files. */
@@ -1361 +1361 @@
             /* Move to next sub-operation. */
             rc = d->pProgress->SetNextOperation(BstrFmt(p->tr("Copy save state file '%s' ..."),
-                                                RTPathFilename(sst.strSaveStateFile.c_str())).raw(), sst.uWeight);
+                                                        RTPathFilename(sst.strSaveStateFile.c_str())).raw(), sst.uWeight);
             if (FAILED(rc)) throw rc;
             /* Copy the file only if it was not copied already. */

trunk/src/VBox/Main/src-server/SnapshotImpl.cpp

@@ -1422 +1422 @@
         && mData->mCurrentSnapshot->i_getDepth() >= SETTINGS_SNAPSHOT_DEPTH_MAX)
     {
-        Utf8Str str;
-        str = "Cannot take another snapshot for machine '%s', because it exceeds the maximum";
-        str += "snapshot depth limit. Please delete some earlier snapshot which you no longer need";
+
         return setError(VBOX_E_INVALID_OBJECT_STATE,
-                        tr(str.c_str()),
+                        tr("Cannot take another snapshot for machine '%s', because it exceeds the maximum snapshot depth limit. Please delete some earlier snapshot which you no longer need"),
                         mUserData->s.strName.c_str());
     }
@@ -2118 +2116 @@
     size_t childrenCount = pSnapshot->i_getChildrenCount();
     if (childrenCount > 1)
-    {
-        str = "Snapshot '%s' of the machine '%s' cannot be deleted, because it has %d child snapshots,";
-        str += "which is more than the one snapshot allowed for deletion";
         return setError(VBOX_E_INVALID_OBJECT_STATE,
-                        tr(str.c_str()),
+                        tr("Snapshot '%s' of the machine '%s' cannot be deleted, because it has %d child snapshots, which is more than the one snapshot allowed for deletion"),
                         pSnapshot->i_getName().c_str(),
                         mUserData->s.strName.c_str(),
                         childrenCount);
-    }
 
     if (pSnapshot == mData->mCurrentSnapshot && childrenCount >= 1)
-    {
-        str = "Snapshot '%s' of the machine '%s' cannot be deleted, because it is the current";
-        str += "snapshot and has one child snapshot";
         return setError(VBOX_E_INVALID_OBJECT_STATE,
-                        tr(str.c_str()),
+                        tr("Snapshot '%s' of the machine '%s' cannot be deleted, because it is the current snapshot and has one child snapshot"),
                         pSnapshot->i_getName().c_str(),
                         mUserData->s.strName.c_str());
-    }
 
     /* If the snapshot being deleted is the current one, ensure current

trunk/src/VBox/Main/src-server/generic/NetIf-generic.cpp

@@ -258 +258 @@
                                                        COM_IIDOF(IHostNetworkInterface),
                                                        HostNetworkInterface::getStaticComponentName(),
-                                                       "Failed to get config info for %s (as reported by '"
-                                                       VBOXNETADPCTL_NAME " add')\n", szBuf);
+                                                       "Failed to get config info for %s (as reported by '" VBOXNETADPCTL_NAME " add')\n", szBuf);
                         }
                         else
@@ -289 +288 @@
                                                COM_IIDOF(IHostNetworkInterface),
                                                HostNetworkInterface::getStaticComponentName(),
-                                               "Failed to execute '" VBOXNETADPCTL_NAME
-                                               " add' (exit status: %d). Check permissions!", rc);
+                                               "Failed to execute '" VBOXNETADPCTL_NAME " add' (exit status: %d). Check permissions!", rc);
                     pclose(fp);
                 }