diff --git a/xwords4/common/comms.c b/xwords4/common/comms.c index 0dbaa0272..8d961cc8a 100644 --- a/xwords4/common/comms.c +++ b/xwords4/common/comms.c @@ -216,6 +216,33 @@ CommsRelayState2Str( CommsRelayState state ) #undef CASE_STR return NULL; } + +const char* +XWREASON2Str( XWREASON reason ) +{ +#define CASE_STR(s) case s: return #s + switch( reason ) { + CASE_STR(XWRELAY_ERROR_NONE); + CASE_STR(XWRELAY_ERROR_OLDFLAGS); + CASE_STR(XWRELAY_ERROR_BADPROTO); + CASE_STR(XWRELAY_ERROR_RELAYBUSY); + CASE_STR(XWRELAY_ERROR_SHUTDOWN); + CASE_STR(XWRELAY_ERROR_TIMEOUT); + CASE_STR(XWRELAY_ERROR_HEART_YOU); + CASE_STR(XWRELAY_ERROR_HEART_OTHER); + CASE_STR(XWRELAY_ERROR_LOST_OTHER); + CASE_STR(XWRELAY_ERROR_OTHER_DISCON); + CASE_STR(XWRELAY_ERROR_NO_ROOM); + CASE_STR(XWRELAY_ERROR_DUP_ROOM); + CASE_STR(XWRELAY_ERROR_TOO_MANY); + CASE_STR(XWRELAY_ERROR_DELETED); + CASE_STR(XWRELAY_ERROR_LASTERR); + default: + XP_ASSERT(0); + } +#undef CASE_STR + return NULL; +} #endif static void @@ -236,7 +263,7 @@ static void init_relay( CommsCtxt* comms, XP_U16 nPlayersHere, XP_U16 nPlayersTotal ) { comms->r.myHostID = comms->isServer? HOST_ID_SERVER: HOST_ID_NONE; - XP_LOGF( "%s: set myHostID to %d", __func__, comms->r.myHostID ); + XP_LOGF( "%s: set hostid: %x", __func__, comms->r.myHostID ); set_relay_state( comms, COMMS_RELAYSTATE_UNCONNECTED ); comms->r.nPlayersHere = nPlayersHere; comms->r.nPlayersTotal = nPlayersTotal; @@ -1173,6 +1200,7 @@ got_connect_cmd( CommsCtxt* comms, XWStreamCtxt* stream, set_relay_state( comms, reconnected ? COMMS_RELAYSTATE_RECONNECTED : COMMS_RELAYSTATE_CONNECTED ); comms->r.myHostID = stream_getU8( stream ); + XP_LOGF( "%s: set hostid: %x", __func__, comms->r.myHostID ); isServer = HOST_ID_SERVER == comms->r.myHostID; if ( isServer != comms->isServer ) { comms->isServer = isServer; @@ -1241,7 +1269,7 @@ relayPreProcess( CommsCtxt* comms, XWStreamCtxt* stream, XWHostID* senderID ) XP_ASSERT( comms->r.myHostID == HOST_ID_NONE || comms->r.myHostID == srcID ); comms->r.myHostID = srcID; - XP_LOGF( "set hostid: %x", comms->r.myHostID ); + XP_LOGF( "%s: set hostid: %x", __func__, comms->r.myHostID ); #ifdef DEBUG { diff --git a/xwords4/common/comms.h b/xwords4/common/comms.h index 902e01504..939acf9b4 100644 --- a/xwords4/common/comms.h +++ b/xwords4/common/comms.h @@ -195,6 +195,7 @@ XP_Bool comms_isConnected( const CommsCtxt* const comms ); void comms_getStats( CommsCtxt* comms, XWStreamCtxt* stream ); const char* ConnType2Str( CommsConnType typ ); const char* CommsRelayState2Str( CommsRelayState state ); +const char* XWREASON2Str( XWREASON reason ); # endif EXTERN_C_END diff --git a/xwords4/linux/cursesmain.c b/xwords4/linux/cursesmain.c index 6ff5f44dc..34ed7f8c0 100644 --- a/xwords4/linux/cursesmain.c +++ b/xwords4/linux/cursesmain.c @@ -1432,9 +1432,11 @@ relay_connd_curses( void* XP_UNUSED(closure), XP_UCHAR* const XP_UNUSED(room), } static void -relay_error_curses( void* XP_UNUSED(closure), XWREASON XP_UNUSED(relayErr) ) +relay_error_curses( void* XP_UNUSED(closure), XWREASON XP_UNUSED_DBG(relayErr) ) { - LOG_FUNC(); +#ifdef DEBUG + XP_LOGF( "%s(%s)", __func__, XWREASON2Str( relayErr ) ); +#endif } void