Fix KBM compile time warnings (#2912)

src/modules/keyboardmanager/common/Helpers.cpp

@@ -159,6 +159,8 @@ namespace KeyboardManagerHelper
             return L"Shortcut cannot have more than one action key";
         case ErrorType::ShortcutMaxShortcutSizeOneActionKey:
             return L"Shortcuts can only have up to 2 modifier keys";
+        default:
+            return L"Unexpected error";
         }
     }
 }

src/modules/keyboardmanager/common/Shortcut.cpp

@@ -99,7 +99,7 @@ DWORD Shortcut::GetWinKey(const ModifierKey& input) const
             //return VK_LWIN by default
             return VK_LWIN;
         }
-        else if (input == ModifierKey::Both)
+        else
         {
             return CommonSharedConstants::VK_WIN_BOTH;
         }