Added default case for GetErrorStr, removed useless const declaration in header

This commit is contained in:
Jack Harper 2019-07-23 23:24:38 +01:00
parent 077eacd419
commit cb455fd410

View File

@ -150,8 +150,8 @@ public:
* This may fail if the username or console ID is already taken. * This may fail if the username or console ID is already taken.
*/ */
void Join(const std::string& nickname, const std::string& console_id_hash, void Join(const std::string& nickname, const std::string& console_id_hash,
const char* server_addr = "127.0.0.1", const u16 server_port = DefaultRoomPort, const char* server_addr = "127.0.0.1", u16 server_port = DefaultRoomPort,
const u16 client_port = 0, const MacAddress& preferred_mac = NoPreferredMac, u16 client_port = 0, const MacAddress& preferred_mac = NoPreferredMac,
const std::string& password = "", const std::string& token = ""); const std::string& password = "", const std::string& token = "");
/** /**
@ -305,8 +305,9 @@ inline const char* GetErrorStr(const RoomMember::Error& e) {
return "PermissionDenied"; return "PermissionDenied";
case RoomMember::Error::NoSuchUser: case RoomMember::Error::NoSuchUser:
return "NoSuchUser"; return "NoSuchUser";
default:
return "Unknown";
} }
return "Unknown";
} }
} // namespace Network } // namespace Network