alway log unhandled commands

Pulse-Eight · Mar 18, 2013 · 3ff78be · 3ff78be
1 parent 58691fb
commit 3ff78be
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/lib/implementations/CECCommandHandler.cpp b/src/lib/implementations/CECCommandHandler.cpp
@@ -756,6 +756,9 @@ void CCECCommandHandler::UnhandledCommand(const cec_command &command, const cec_
   {
     LIB_CEC->AddLog(CEC_LOG_DEBUG, "sending abort with opcode %02x and reason '%s' to %s", command.opcode, ToString(reason), ToString(command.initiator));
     m_processor->TransmitAbort(command.destination, command.initiator, command.opcode, reason);
+
+    if (reason == CEC_ABORT_REASON_INVALID_OPERAND)
+      RequestEmailFromCustomer(command);
   }
 }
 

diff --git a/src/lib/implementations/RLCommandHandler.cpp b/src/lib/implementations/RLCommandHandler.cpp
@@ -108,6 +108,7 @@ int CRLCommandHandler::HandleDeviceVendorCommandWithId(const cec_command &comman
     case CEC_OPCODE_USER_CONTROL_PRESSED:
       if (command.parameters.size == 5)
       {
+        bHandled = true;
         switch (command.parameters[4])
         {
         // top menu -> root menu
@@ -119,10 +120,9 @@ int CRLCommandHandler::HandleDeviceVendorCommandWithId(const cec_command &comman
           client->SetCurrentButton(CEC_USER_CONTROL_CODE_CONTENTS_MENU);
           break;
         default:
-          RequestEmailFromCustomer(command);
+          bHandled = false;
           break;
         }
-        bHandled = true;
       }
       break;
     // user control released