Remove printfs from qpTestLog
authorJamie Madill <jmadill@chromium.org>
Thu, 20 Apr 2017 19:11:08 +0000 (15:11 -0400)
committerPyry Haulos <phaulos@google.com>
Thu, 20 Apr 2017 19:40:11 +0000 (19:40 +0000)
These clutter up the standard output with shader compile times.
Also remove qpTestLog_writeMessage since it didn't seem to be used.

Change-Id: I218651f37ff6c9f40f6e81115f7fc64adb8d8500

framework/qphelper/qpTestLog.c
framework/qphelper/qpTestLog.h

index 5e60797..00603c1 100644 (file)
@@ -551,31 +551,6 @@ static deBool qpTestLog_writeKeyValuePair (qpTestLog* log, const char* elementNa
 }
 
 /*--------------------------------------------------------------------*//*!
- * \brief Write a message to output log
- * \param log          qpTestLog instance
- * \param format       Format string of message
- * \param ...          Parameters for message
- * \return true if ok, false otherwise
- *//*--------------------------------------------------------------------*/
-deBool qpTestLog_writeMessage (qpTestLog* log, const char* format, ...)
-{
-       char    buffer[1024];
-       va_list args;
-
-       /* \todo [petri] Handle buffer overflows! */
-
-       va_start(args, format);
-       buffer[DE_LENGTH_OF_ARRAY(buffer) - 1] = 0;
-       vsnprintf(buffer, sizeof(buffer), format, args);
-       va_end(args);
-
-       printf("%s\n", buffer);
-
-       /* <Text>text</Text> */
-       return qpTestLog_writeKeyValuePair(log, "Text", DE_NULL, DE_NULL, DE_NULL, QP_KEY_TAG_LAST, buffer);
-}
-
-/*--------------------------------------------------------------------*//*!
  * \brief Write key-value-pair into log
  * \param log                  qpTestLog instance
  * \param name                 Unique identifier for entry
@@ -604,8 +579,6 @@ deBool qpTestLog_writeInteger (qpTestLog* log, const char* name, const char* des
        char tmpString[64];
        int64ToString(value, tmpString);
 
-       printf("%s = %lld %s\n", description, (signed long long)value, unit ? unit : "");
-
        /* <Number Name="name" Description="description" Tag="Performance">15</Number> */
        return qpTestLog_writeKeyValuePair(log, "Number", name, description, unit, tag, tmpString);
 }
@@ -624,8 +597,6 @@ deBool qpTestLog_writeFloat (qpTestLog* log, const char* name, const char* descr
        char tmpString[64];
        floatToString(value, tmpString, sizeof(tmpString));
 
-       printf("%s = %f %s\n", description, value, unit ? unit : "");
-
        /* <Number Name="name" Description="description" Tag="Performance">15</Number> */
        return qpTestLog_writeKeyValuePair(log, "Number", name, description, unit, tag, tmpString);
 }
index 807716e..d57f3c1 100644 (file)
@@ -195,7 +195,6 @@ deBool                      qpTestLog_startCase                             (qpTestLog* log, const char* testCasePath, qpTes
 deBool                 qpTestLog_endCase                               (qpTestLog* log, qpTestResult result, const char* description);
 deBool                 qpTestLog_terminateCase                 (qpTestLog* log, qpTestResult result);
 
-deBool                 qpTestLog_writeMessage                  (qpTestLog* log, const char* format, ...) DE_PRINTF_FUNC_ATTR(2,3);
 deBool                 qpTestLog_startSection                  (qpTestLog* log, const char* name, const char* description);
 deBool                 qpTestLog_endSection                    (qpTestLog* log);
 deBool                 qpTestLog_writeText                             (qpTestLog* log, const char* name, const char* description, qpKeyValueTag tag, const char* value);