Refactor to not return from middle of function. No behavior change.

This commit is contained in:
ehouse 2006-11-11 22:38:20 +00:00
parent e2f3a55611
commit 130c0f5a92

View file

@ -472,6 +472,7 @@ callTurnChangeListener( ServerCtxt* server )
static XP_Bool
handleRegistrationMsg( ServerCtxt* server, XWStreamCtxt* stream )
{
XP_Bool success = XP_TRUE;
XP_U16 playersInMsg, i;
XP_STATUSF( "handleRegistrationMsg" );
@ -481,9 +482,8 @@ handleRegistrationMsg( ServerCtxt* server, XWStreamCtxt* stream )
if ( server->nv.pendingRegistrations < playersInMsg ) {
util_userError( server->vol.util, ERR_REG_UNEXPECTED_USER );
return XP_FALSE;
}
success = XP_FALSE;
} else {
for ( i = 0; i < playersInMsg; ++i ) {
registerRemotePlayer( server, stream );
@ -498,7 +498,8 @@ handleRegistrationMsg( ServerCtxt* server, XWStreamCtxt* stream )
assignTilesToAll( server );
SETSTATE( server, XWSTATE_RECEIVED_ALL_REG );
}
return XP_TRUE;
}
return success;
} /* handleRegistrationMsg */
#endif