Index: /trunk/src/VBox/Runtime/r3/test.cpp
===================================================================
--- /trunk/src/VBox/Runtime/r3/test.cpp	(revision 44139)
+++ /trunk/src/VBox/Runtime/r3/test.cpp	(revision 44140)
@@ -186,6 +186,4 @@
 static void rtTestXmlElemStartV(PRTTESTINT pTest, const char *pszTag, const char *pszAttrFmt, va_list va);
 static void rtTestXmlElemStart(PRTTESTINT pTest, const char *pszTag, const char *pszAttrFmt, ...);
-static void rtTestXmlElemValueV(PRTTESTINT pTest, const char *pszFormat, va_list va);
-static void rtTestXmlElemValue(PRTTESTINT pTest, const char *pszFormat, ...);
 static void rtTestXmlElemEnd(PRTTESTINT pTest, const char *pszTag);
 static void rtTestXmlEnd(PRTTESTINT pTest);
@@ -794,41 +792,4 @@
 
 /**
- * Writes an element value, or a part of one, taking care of all the escaping.
- *
- * The caller must own the instance lock.
- *
- * @param   pTest               The test instance.
- * @param   pszFormat           The value format string.
- * @param   va                  The format arguments.
- */
-static void rtTestXmlElemValueV(PRTTESTINT pTest, const char *pszFormat, va_list va)
-{
-    if (pTest->fXmlEnabled)
-    {
-        char *pszValue;
-        RTStrAPrintfV(&pszValue, pszFormat, va);
-        if (pszValue)
-        {
-            rtTestXmlOutput(pTest, "%RMes", pszValue);
-            RTStrFree(pszValue);
-        }
-        pTest->eXmlState = RTTESTINT::kXmlPos_Value;
-    }
-}
-
-
-/**
- * Wrapper around rtTestXmlElemValueV.
- */
-static void rtTestXmlElemValue(PRTTESTINT pTest, const char *pszFormat, ...)
-{
-    va_list va;
-    va_start(va, pszFormat);
-    rtTestXmlElemValueV(pTest, pszFormat, va);
-    va_end(va);
-}
-
-
-/**
  * Ends the current element.
  *
@@ -1130,5 +1091,5 @@
         {
             rtTestXmlElem(pTest, "Passed", NULL);
-            rtTestXmlElemEnd(pTest, "SubTest");
+            rtTestXmlElemEnd(pTest, "Test");
             cch += RTTestPrintfNl(pTest, RTTESTLVL_SUB_TEST, "%-50s: PASSED\n", pTest->pszSubTest);
         }
@@ -1137,5 +1098,5 @@
             pTest->cSubTestsFailed++;
             rtTestXmlElem(pTest, "Failed", "errors=\"%u\"", cErrors);
-            rtTestXmlElemEnd(pTest, "SubTest");
+            rtTestXmlElemEnd(pTest, "Test");
             cch += RTTestPrintfNl(pTest, RTTESTLVL_SUB_TEST, "%-50s: FAILED (%u errors)\n",
                                   pTest->pszSubTest, cErrors);
@@ -1272,5 +1233,5 @@
         cch = RTTestPrintfNl(hTest, RTTESTLVL_DEBUG, "debug: Starting sub-test '%s'\n", pszSubTest);
 
-    rtTestXmlElemStart(pTest, "SubTest", "name=%RMas", pszSubTest);
+    rtTestXmlElemStart(pTest, "Test", "name=%RMas", pszSubTest);
 
     RTCritSectLeave(&pTest->Lock);
@@ -1456,7 +1417,5 @@
 
     RTCritSectEnter(&pTest->Lock);
-    rtTestXmlElemStart(pTest, "Value", "name=%RMas unit=%RMas", pszName, pszUnit);
-    rtTestXmlElemValue(pTest, "%llu", u64Value);
-    rtTestXmlElemEnd(pTest, "Value");
+    rtTestXmlElem(pTest, "Value", "name=%RMas unit=%RMas value=\"%llu\"", pszName, pszUnit, u64Value);
     RTCritSectLeave(&pTest->Lock);