diff --git a/src/game/server/ddracechat.cpp b/src/game/server/ddracechat.cpp index cd60ba73b..43f94dbad 100644 --- a/src/game/server/ddracechat.cpp +++ b/src/game/server/ddracechat.cpp @@ -1272,6 +1272,8 @@ void CGameContext::ConTime(IConsole::IResult *pResult, void *pUserData) pSelf->SendBroadcast(aBuftime, pResult->m_ClientID); } +static const char s_aaMsg[3][128] = {"game/round timer.", "broadcast.", "both game/round timer and broadcast."}; + void CGameContext::ConSetTimerType(IConsole::IResult *pResult, void *pUserData) { CGameContext *pSelf = (CGameContext *) pUserData; @@ -1283,60 +1285,65 @@ void CGameContext::ConSetTimerType(IConsole::IResult *pResult, void *pUserData) if (!pPlayer) return; - const char msg[3][128] = {"game/round timer.", "broadcast.", "both game/round timer and broadcast."}; char aBuf[128]; - if(pPlayer->m_TimerType <= CPlayer::TIMERTYPE_GAMETIMER_AND_BROADCAST && pPlayer->m_TimerType >= CPlayer::TIMERTYPE_GAMETIMER) - str_format(aBuf, sizeof(aBuf), "Timer is displayed in %s", msg[pPlayer->m_TimerType]); - else if(pPlayer->m_TimerType == CPlayer::TIMERTYPE_NONE) - str_format(aBuf, sizeof(aBuf), "Timer isn't displayed."); - - int OldType = pPlayer->m_TimerType; - - if(pResult->NumArguments() == 0) { - pSelf->Console()->Print(IConsole::OUTPUT_LEVEL_STANDARD,"timer",aBuf); - return; - } - else if(str_comp_nocase(pResult->GetString(0), "gametimer") == 0) + + if(pResult->NumArguments() > 0) { - if(pPlayer->m_ClientVersion >= VERSION_DDNET_GAMETICK) - pPlayer->m_TimerType = CPlayer::TIMERTYPE_GAMETIMER; - else - pSelf->Console()->Print(IConsole::OUTPUT_LEVEL_STANDARD, "timer", "gametimer is not supported by your client."); - } - else if(str_comp_nocase(pResult->GetString(0), "broadcast") == 0) - pPlayer->m_TimerType = CPlayer::TIMERTYPE_BROADCAST; - else if(str_comp_nocase(pResult->GetString(0), "both") == 0) - { - if(pPlayer->m_ClientVersion >= VERSION_DDNET_GAMETICK) - pPlayer->m_TimerType = CPlayer::TIMERTYPE_GAMETIMER_AND_BROADCAST; - else + int OldType = pPlayer->m_TimerType; + + if(str_comp_nocase(pResult->GetString(0), "gametimer") == 0) { - pPlayer->m_TimerType = CPlayer::TIMERTYPE_BROADCAST; - pSelf->Console()->Print(IConsole::OUTPUT_LEVEL_STANDARD, "timer", "gametimer is not supported by your client."); - } - } - else if(str_comp_nocase(pResult->GetString(0), "none") == 0) - pPlayer->m_TimerType = CPlayer::TIMERTYPE_NONE; - else if(str_comp_nocase(pResult->GetString(0), "cycle") == 0) - { - if(pPlayer->m_ClientVersion >= VERSION_DDNET_GAMETICK) - { - if(pPlayer->m_TimerType < CPlayer::TIMERTYPE_NONE) - pPlayer->m_TimerType++; - else if(pPlayer->m_TimerType == CPlayer::TIMERTYPE_NONE) + if(pPlayer->m_ClientVersion >= VERSION_DDNET_GAMETICK) pPlayer->m_TimerType = CPlayer::TIMERTYPE_GAMETIMER; + else + pSelf->Console()->Print(IConsole::OUTPUT_LEVEL_STANDARD, "timer", "gametimer is not supported by your client."); + } + else if(str_comp_nocase(pResult->GetString(0), "broadcast") == 0) + pPlayer->m_TimerType = CPlayer::TIMERTYPE_BROADCAST; + else if(str_comp_nocase(pResult->GetString(0), "both") == 0) + { + if(pPlayer->m_ClientVersion >= VERSION_DDNET_GAMETICK) + pPlayer->m_TimerType = CPlayer::TIMERTYPE_GAMETIMER_AND_BROADCAST; + else + { + pPlayer->m_TimerType = CPlayer::TIMERTYPE_BROADCAST; + pSelf->Console()->Print(IConsole::OUTPUT_LEVEL_STANDARD, "timer", "gametimer is not supported by your client."); + } + } + else if(str_comp_nocase(pResult->GetString(0), "none") == 0) + pPlayer->m_TimerType = CPlayer::TIMERTYPE_NONE; + else if(str_comp_nocase(pResult->GetString(0), "cycle") == 0) + { + if(pPlayer->m_ClientVersion >= VERSION_DDNET_GAMETICK) + { + if(pPlayer->m_TimerType < CPlayer::TIMERTYPE_NONE) + pPlayer->m_TimerType++; + else if(pPlayer->m_TimerType == CPlayer::TIMERTYPE_NONE) + pPlayer->m_TimerType = CPlayer::TIMERTYPE_GAMETIMER; + } + else + { + if(pPlayer->m_TimerType < CPlayer::TIMERTYPE_NONE) + pPlayer->m_TimerType = CPlayer::TIMERTYPE_NONE; + else if(pPlayer->m_TimerType == CPlayer::TIMERTYPE_NONE) + pPlayer->m_TimerType = CPlayer::TIMERTYPE_BROADCAST; + } } else { - if(pPlayer->m_TimerType < CPlayer::TIMERTYPE_NONE) - pPlayer->m_TimerType = CPlayer::TIMERTYPE_NONE; - else if(pPlayer->m_TimerType == CPlayer::TIMERTYPE_NONE) - pPlayer->m_TimerType = CPlayer::TIMERTYPE_BROADCAST; + str_format(aBuf, sizeof(aBuf), "Unknown \"%s\" parameter. Accepted values: gametimer, broadcast, both, none, cycle", pResult->GetString(0)); + pSelf->Console()->Print(IConsole::OUTPUT_LEVEL_STANDARD, "timer", aBuf); + return; } + + if((OldType == CPlayer::TIMERTYPE_BROADCAST || OldType == CPlayer::TIMERTYPE_GAMETIMER_AND_BROADCAST) && (pPlayer->m_TimerType == CPlayer::TIMERTYPE_GAMETIMER || pPlayer->m_TimerType == CPlayer::TIMERTYPE_NONE)) + pSelf->SendBroadcast("", pResult->m_ClientID); } - if((OldType == CPlayer::TIMERTYPE_BROADCAST || OldType == CPlayer::TIMERTYPE_GAMETIMER_AND_BROADCAST) && (pPlayer->m_TimerType == CPlayer::TIMERTYPE_GAMETIMER || pPlayer->m_TimerType == CPlayer::TIMERTYPE_NONE)) - pSelf->SendBroadcast("", pResult->m_ClientID); + if(pPlayer->m_TimerType <= CPlayer::TIMERTYPE_GAMETIMER_AND_BROADCAST && pPlayer->m_TimerType >= CPlayer::TIMERTYPE_GAMETIMER) + str_format(aBuf, sizeof(aBuf), "Timer is displayed in %s", s_aaMsg[pPlayer->m_TimerType]); + else if(pPlayer->m_TimerType == CPlayer::TIMERTYPE_NONE) + str_format(aBuf, sizeof(aBuf), "Timer isn't displayed."); pSelf->Console()->Print(IConsole::OUTPUT_LEVEL_STANDARD, "timer", aBuf); }