cec: show the command as string instead of int when an unexpected command was received.
authorLars Op den Kamp <lars@opdenkamp.eu>
Mon, 31 Oct 2011 19:55:59 +0000 (20:55 +0100)
committerLars Op den Kamp <lars@opdenkamp.eu>
Mon, 31 Oct 2011 19:55:59 +0000 (20:55 +0100)
src/lib/AdapterCommunication.cpp
src/lib/AdapterCommunication.h
src/lib/CECProcessor.cpp

index 9d62afef1049457225502000ebab854c4a66fb21..122565ee4093b87e6fddf618e9479a1afdf991cf 100644 (file)
@@ -89,7 +89,7 @@ CCECAdapterMessage &CCECAdapterMessage::operator =(const CCECAdapterMessage &msg
   return *this;
 }
 
-CStdString CCECAdapterMessage::ToString(void) const
+CStdString CCECAdapterMessage::MessageCodeAsString(void) const
 {
   CStdString strMsg;
   switch (message())
@@ -101,31 +101,19 @@ CStdString CCECAdapterMessage::ToString(void) const
     strMsg = "PING";
     break;
   case MSGCODE_TIMEOUT_ERROR:
+    strMsg = "TIMEOUT";
+    break;
   case MSGCODE_HIGH_ERROR:
+    strMsg = "HIGH_ERROR";
+    break;
   case MSGCODE_LOW_ERROR:
-    {
-      if (message() == MSGCODE_TIMEOUT_ERROR)
-        strMsg = "TIMEOUT";
-      else if (message() == MSGCODE_HIGH_ERROR)
-        strMsg = "HIGH_ERROR";
-      else
-        strMsg = "LOW_ERROR";
-
-      int iLine      = (size() >= 3) ? (at(1) << 8) | at(2) : 0;
-      uint32_t iTime = (size() >= 7) ? (at(3) << 24) | (at(4) << 16) | (at(5) << 8) | at(6) : 0;
-      strMsg.AppendFormat(" line:%i", iLine);
-      strMsg.AppendFormat(" time:%u", iTime);
-    }
+    strMsg = "LOW_ERROR";
     break;
   case MSGCODE_FRAME_START:
     strMsg = "FRAME_START";
-    if (size() >= 2)
-      strMsg.AppendFormat(" initiator:%1x destination:%1x ack:%s %s", initiator(), destination(), ack() ? "high" : "low", eom() ? "eom" : "");
     break;
   case MSGCODE_FRAME_DATA:
     strMsg = "FRAME_DATA";
-    if (size() >= 2)
-      strMsg.AppendFormat(" %02x %s", at(1), eom() ? "eom" : "");
     break;
   case MSGCODE_RECEIVE_FAILED:
     strMsg = "RECEIVE_FAILED";
@@ -186,6 +174,38 @@ CStdString CCECAdapterMessage::ToString(void) const
   return strMsg;
 }
 
+CStdString CCECAdapterMessage::ToString(void) const
+{
+  CStdString strMsg;
+  strMsg = MessageCodeAsString();
+
+  switch (message())
+  {
+  case MSGCODE_TIMEOUT_ERROR:
+  case MSGCODE_HIGH_ERROR:
+  case MSGCODE_LOW_ERROR:
+    {
+      int iLine      = (size() >= 3) ? (at(1) << 8) | at(2) : 0;
+      uint32_t iTime = (size() >= 7) ? (at(3) << 24) | (at(4) << 16) | (at(5) << 8) | at(6) : 0;
+      strMsg.AppendFormat(" line:%i", iLine);
+      strMsg.AppendFormat(" time:%u", iTime);
+    }
+    break;
+  case MSGCODE_FRAME_START:
+    if (size() >= 2)
+      strMsg.AppendFormat(" initiator:%1x destination:%1x ack:%s %s", initiator(), destination(), ack() ? "high" : "low", eom() ? "eom" : "");
+    break;
+  case MSGCODE_FRAME_DATA:
+    if (size() >= 2)
+      strMsg.AppendFormat(" %02x %s", at(1), eom() ? "eom" : "");
+    break;
+  default:
+    break;
+  }
+
+  return strMsg;
+}
+
 bool CCECAdapterMessage::is_error(void) const
 {
   cec_adapter_messagecode code = message();
index 91e5ee64a0de2a283e2af2832d01334e45b886ea..21a775dbc6b8739b87f0982041fbdd168182b300 100644 (file)
@@ -58,6 +58,7 @@ namespace CEC
     CCECAdapterMessage(const cec_command &command);
     CCECAdapterMessage &operator =(const CCECAdapterMessage &msg);
     CStdString ToString(void) const;
+    CStdString MessageCodeAsString(void) const;
 
     bool                    empty(void) const             { return packet.empty(); }
     uint8_t                 operator[](uint8_t pos) const { return packet[pos]; }
index 3b6f6f2e1cac5a18b7d810135e3a7d1116921799..5568d9506a999807407cfc3095187c91f54d9d24 100644 (file)
@@ -316,7 +316,7 @@ bool CCECProcessor::WaitForTransmitSucceeded(uint8_t iLength, uint32_t iTimeout
       break;
     default:
       CStdString strLog;
-      strLog.Format("received unexpected reply '%1x' instead of ack", msg.message());
+      strLog.Format("received unexpected reply '%s' instead of ack", msg.MessageCodeAsString().c_str());
       m_controller->AddLog(CEC_LOG_WARNING, strLog);
       bError = true;
       break;