Some respect for OOP. (MasterPlayerNumber)

src/AutoKeysounds.cpp

@@ -49,7 +49,7 @@ void AutoKeysounds::LoadAutoplaySoundsInto( RageSoundReader_Chain *pChain )
 	 * Add all current autoplay sounds in both players to the chain.  If a sound is
 	 * common to both players, don't pan it; otherwise pan it to that player's side.
 	 */
-	int iNumTracks = m_ndAutoKeysoundsOnly[GAMESTATE->m_MasterPlayerNumber].GetNumTracks();
+	int iNumTracks = m_ndAutoKeysoundsOnly[GAMESTATE->GetMasterPlayerNumber()].GetNumTracks();
 	for( int t = 0; t < iNumTracks; t++ )
 	{
 		int iRow = -1;
@@ -297,7 +297,7 @@ void AutoKeysounds::FinishLoading()
 		apSounds.push_back( m_pPlayerSounds[1] );
 	}
 
-	if( GAMESTATE->GetNumPlayersEnabled() == 1 && GAMESTATE->m_MasterPlayerNumber == PLAYER_2 )
+	if( GAMESTATE->GetNumPlayersEnabled() == 1 && GAMESTATE->GetMasterPlayerNumber() == PLAYER_2 )
 		swap( m_pPlayerSounds[PLAYER_1], m_pPlayerSounds[PLAYER_2] );
 
 	if( apSounds.size() > 1 )

src/CourseContentsList.cpp

@@ -42,9 +42,9 @@ void CourseContentsList::SetFromGameState()
 {
 	RemoveAllChildren();
 
-	if( GAMESTATE->m_MasterPlayerNumber == PlayerNumber_Invalid )
+	if( GAMESTATE->GetMasterPlayerNumber() == PlayerNumber_Invalid )
 		return;
-	const Trail *pMasterTrail = GAMESTATE->m_pCurTrail[GAMESTATE->m_MasterPlayerNumber];
+	const Trail *pMasterTrail = GAMESTATE->m_pCurTrail[GAMESTATE->GetMasterPlayerNumber()];
 	if( pMasterTrail == NULL )
 		return;
 	unsigned uNumEntriesToShow = pMasterTrail->m_vEntries.size(); 

src/GameState.cpp

@@ -190,6 +190,16 @@ GameState::~GameState()
 	SAFE_DELETE( g_pImpl );
 }
 
+PlayerNumber GameState::GetMasterPlayerNumber() const
+{
+	return this->masterPlayerNumber;
+}
+
+void GameState::SetMasterPlayerNumber(const PlayerNumber p)
+{
+	this->masterPlayerNumber = p;
+}
+
 void GameState::ApplyGameCommand( const RString &sCommand, PlayerNumber pn )
 {
 	GameCommand m;
@@ -244,7 +254,7 @@ void GameState::ResetPlayer( PlayerNumber pn )
 
 void GameState::Reset()
 {
-	m_MasterPlayerNumber = PLAYER_INVALID; // must initialize for UnjoinPlayer
+	this->SetMasterPlayerNumber(PLAYER_INVALID); // must initialize for UnjoinPlayer
 
 	FOREACH_PlayerNumber( pn )
 		UnjoinPlayer( pn );
@@ -334,14 +344,14 @@ void GameState::JoinPlayer( PlayerNumber pn )
 	 * give the new player the same number of stage tokens that the old player
 	 * has. */
 	if( GetCoinMode() == CoinMode_Pay && GetPremium() == Premium_2PlayersFor1Credit && GetNumSidesJoined() == 1 )
-		m_iPlayerStageTokens[pn] = m_iPlayerStageTokens[m_MasterPlayerNumber];
+		m_iPlayerStageTokens[pn] = m_iPlayerStageTokens[this->GetMasterPlayerNumber()];
 	else
 		m_iPlayerStageTokens[pn] = PREFSMAN->m_iSongsPerPlay;
 
 	m_bSideIsJoined[pn] = true;
 
-	if( m_MasterPlayerNumber == PLAYER_INVALID )
+	if( this->GetMasterPlayerNumber() == PLAYER_INVALID )
-		m_MasterPlayerNumber = pn;
+		this->SetMasterPlayerNumber(pn);
 
 	// if first player to join, set start time
 	if( GetNumSidesJoined() == 1 )
@@ -385,8 +395,8 @@ void GameState::UnjoinPlayer( PlayerNumber pn )
 
 	ResetPlayer( pn );
 
-	if( m_MasterPlayerNumber == pn )
+	if( this->GetMasterPlayerNumber() == pn )
-		m_MasterPlayerNumber = GetFirstHumanPlayer();
+		this->SetMasterPlayerNumber(GetFirstHumanPlayer());
 
 	/* Unjoin STATSMAN first, so steps used by this player are released
 	 * and can be released by PROFILEMAN. */
@@ -398,7 +408,7 @@ void GameState::UnjoinPlayer( PlayerNumber pn )
 	MESSAGEMAN->Broadcast( msg );
 
 	// If there are no players left, reset some non-player-specific stuff, too.
-	if( m_MasterPlayerNumber == PLAYER_INVALID )
+	if( this->GetMasterPlayerNumber() == PLAYER_INVALID )
 	{
 		SongOptions so;
 		GetDefaultSongOptions( so );
@@ -635,8 +645,8 @@ int GameState::GetNumStagesForCurrentSongAndStepsOrCourse() const
 		if( pStyle == NULL )
 		{
 			const Steps *pSteps = NULL;
-			if( m_MasterPlayerNumber != PlayerNumber_Invalid )
+			if( this->GetMasterPlayerNumber() != PlayerNumber_Invalid )
-				pSteps = m_pCurSteps[m_MasterPlayerNumber];
+				pSteps = m_pCurSteps[this->GetMasterPlayerNumber()];
 			// Don't call GetFirstCompatibleStyle if numSidesJoined == 0.
 			// This happens because on SContinue when players are unjoined,
 			// pCurSteps will still be set while no players are joined. -Chris
@@ -994,9 +1004,9 @@ bool GameState::IsFinalStageForAnyHumanPlayer() const
 
 bool GameState::IsAnExtraStage() const
 {
-	if( m_MasterPlayerNumber == PlayerNumber_Invalid )
+	if( this->GetMasterPlayerNumber() == PlayerNumber_Invalid )
 		return false;
-	return !IsEventMode() && !IsCourseMode() && m_iAwardedExtraStages[m_MasterPlayerNumber] > 0;
+	return !IsEventMode() && !IsCourseMode() && m_iAwardedExtraStages[this->GetMasterPlayerNumber()] > 0;
 }
 
 static ThemeMetric<bool> LOCK_EXTRA_STAGE_SELECTION("GameState","LockExtraStageSelection");
@@ -1007,16 +1017,16 @@ bool GameState::IsAnExtraStageAndSelectionLocked() const
 
 bool GameState::IsExtraStage() const
 {
-	if( m_MasterPlayerNumber == PlayerNumber_Invalid )
+	if( this->GetMasterPlayerNumber() == PlayerNumber_Invalid )
 		return false;
-	return !IsEventMode() && !IsCourseMode() && m_iAwardedExtraStages[m_MasterPlayerNumber] == 1;
+	return !IsEventMode() && !IsCourseMode() && m_iAwardedExtraStages[this->GetMasterPlayerNumber()] == 1;
 }
 
 bool GameState::IsExtraStage2() const
 {
-	if( m_MasterPlayerNumber == PlayerNumber_Invalid )
+	if( this->GetMasterPlayerNumber() == PlayerNumber_Invalid )
 		return false;
-	return !IsEventMode() && !IsCourseMode() && m_iAwardedExtraStages[m_MasterPlayerNumber] == 2;
+	return !IsEventMode() && !IsCourseMode() && m_iAwardedExtraStages[this->GetMasterPlayerNumber()] == 2;
 }
 
 Stage GameState::GetCurrentStage() const
@@ -1059,7 +1069,7 @@ int GameState::GetCourseSongIndex() const
 	}
 	else
 	{
-		return STATSMAN->m_CurStageStats.m_player[m_MasterPlayerNumber].m_iSongsPlayed-1;
+		return STATSMAN->m_CurStageStats.m_player[this->GetMasterPlayerNumber()].m_iSongsPlayed-1;
 	}
 }
 
@@ -1131,7 +1141,7 @@ void GameState::SetCurrentStyle( const Style *pStyle )
 	if( INPUTMAPPER )
 	{
 		if( GetCurrentStyle() && GetCurrentStyle()->m_StyleType == StyleType_OnePlayerTwoSides )
-			INPUTMAPPER->SetJoinControllers( m_MasterPlayerNumber );
+			INPUTMAPPER->SetJoinControllers( this->GetMasterPlayerNumber() );
 		else
 			INPUTMAPPER->SetJoinControllers( PLAYER_INVALID );
 	}
@@ -1192,7 +1202,7 @@ bool GameState::IsHumanPlayer( PlayerNumber pn ) const
 		return true;
 	case StyleType_OnePlayerOneSide:
 	case StyleType_OnePlayerTwoSides:
-		return pn == m_MasterPlayerNumber;
+		return pn == this->GetMasterPlayerNumber();
 	default:
 		ASSERT(0);		// invalid style type
 		return false;
@@ -1276,7 +1286,7 @@ EarnedExtraStage GameState::CalculateEarnedExtraStage() const
 	if( GetSmallestNumStagesLeftForAnyHumanPlayer() > 0 )
 		return EarnedExtraStage_No;
 
-	if( m_iAwardedExtraStages[m_MasterPlayerNumber] >= 2 )
+	if( m_iAwardedExtraStages[this->GetMasterPlayerNumber()] >= 2 )
 		return EarnedExtraStage_No;
 
 	FOREACH_EnabledPlayer( pn )
@@ -2124,7 +2134,7 @@ public:
 	DEFINE_METHOD( IsPlayerEnabled,			IsPlayerEnabled(Enum::Check<PlayerNumber>(L, 1)) )
 	DEFINE_METHOD( IsHumanPlayer,			IsHumanPlayer(Enum::Check<PlayerNumber>(L, 1)) )
 	DEFINE_METHOD( GetPlayerDisplayName,		GetPlayerDisplayName(Enum::Check<PlayerNumber>(L, 1)) )
-	DEFINE_METHOD( GetMasterPlayerNumber,		m_MasterPlayerNumber )
+	DEFINE_METHOD( GetMasterPlayerNumber,		GetMasterPlayerNumber() )
 	DEFINE_METHOD( GetMultiplayer,			m_bMultiplayer )
 	static int SetMultiplayer( T* p, lua_State *L )
 	{

src/GameState.h

@@ -35,6 +35,8 @@ class Trail;
 /** @brief Holds game data that is not saved between sessions. */
 class GameState
 {
+	/** @brief The player number used with Styles where one player controls both sides. */
+	PlayerNumber	masterPlayerNumber;
 public:
 	/** @brief Set up the GameState with initial values. */
 	GameState();
@@ -91,8 +93,6 @@ public:
 	 * to get one credit, only to have to put in another four coins to get
 	 * the three credits needed to begin the game. */
 	BroadcastOnChange<int>			m_iCoins;
-	/** @brief The player number used with Styles where one player controls both sides. */
-	PlayerNumber	m_MasterPlayerNumber;
 	bool			m_bMultiplayer;
 	int				m_iNumMultiplayerNoteFields;
 	bool DifficultiesLocked() const;
@@ -143,10 +143,12 @@ public:
 	/**
 	 * @brief Retrieve the present master player number.
 	 * @return The master player number. */
-	PlayerNumber GetMasterPlayerNumber() const
+	PlayerNumber GetMasterPlayerNumber() const;
-	{
+	
-		return m_MasterPlayerNumber;
+	/**
-	}
+	 * @brief Set the master player number.
+	 * @param p the master player number. */
+	void SetMasterPlayerNumber(const PlayerNumber p);
 
 	bool IsCourseMode() const;
 	bool IsBattleMode() const; // not Rave

src/NoteField.cpp

@@ -303,7 +303,7 @@ void NoteField::Update( float fDeltaTime )
 	// TODO: Remove use of PlayerNumber.
 
 	PlayerNumber pn = m_pPlayerState->m_PlayerNumber;
-	if( pn == GAMESTATE->m_MasterPlayerNumber )
+	if( pn == GAMESTATE->GetMasterPlayerNumber() )
 		NoteDisplay::Update( fDeltaTime );
 }
 

src/OptionRow.cpp

@@ -196,7 +196,7 @@ RString OptionRow::GetRowTitle() const
 
 		if( GAMESTATE->m_pCurCourse )
 		{
-			const Trail* pTrail = GAMESTATE->m_pCurTrail[GAMESTATE->m_MasterPlayerNumber];
+			const Trail* pTrail = GAMESTATE->m_pCurTrail[GAMESTATE->GetMasterPlayerNumber()];
 			ASSERT( pTrail != NULL );
 			const int iNumCourseEntries = pTrail->m_vEntries.size();
 			if( iNumCourseEntries > CommonMetrics::MAX_COURSE_ENTRIES_BEFORE_VARIOUS )
@@ -417,8 +417,8 @@ void OptionRow::AfterImportOptions( PlayerNumber pn )
 	// we need to copy p2 to p1, not p1 to p2.
 	if( m_pHand->m_Def.m_bOneChoiceForAllPlayers )
 	{
-		PlayerNumber pnCopyFrom = GAMESTATE->m_MasterPlayerNumber;
+		PlayerNumber pnCopyFrom = GAMESTATE->GetMasterPlayerNumber();
-		if( GAMESTATE->m_MasterPlayerNumber == PLAYER_INVALID )
+		if( GAMESTATE->GetMasterPlayerNumber() == PLAYER_INVALID )
 			pnCopyFrom = PLAYER_1;
 		FOREACH_PlayerNumber( p )
 			m_vbSelected[p] = m_vbSelected[pnCopyFrom];

src/Player.cpp

@@ -1375,7 +1375,7 @@ void Player::DrawPrimitives()
 
 	// May have both players in doubles (for battle play); only draw primary player.
 	if( GAMESTATE->GetCurrentStyle()->m_StyleType == StyleType_OnePlayerTwoSides  &&
-		pn != GAMESTATE->m_MasterPlayerNumber )
+		pn != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 
 	// Draw these below everything else.

src/PlayerState.cpp

@@ -162,7 +162,7 @@ void PlayerState::RebuildPlayerOptionsFromActiveAttacks()
 		so.FromString( m_ActiveAttacks[s].sModifiers );
 	}
 	m_PlayerOptions.Assign( ModsLevel_Song, po );
-	if( m_PlayerNumber == GAMESTATE->m_MasterPlayerNumber )
+	if( m_PlayerNumber == GAMESTATE->GetMasterPlayerNumber() )
 		GAMESTATE->m_SongOptions.Assign( ModsLevel_Song, so );
 
 	int iSumOfAttackLevels = GetSumOfActiveAttackLevels();

src/ScoreKeeperShared.cpp

@@ -16,7 +16,7 @@ void ScoreKeeperShared::Load(
 	const vector<Steps*> &apSteps,
 	const vector<AttackArray> &asModifiers )
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::Load( apSongs, apSteps, asModifiers );
 }
@@ -24,63 +24,63 @@ void ScoreKeeperShared::Load(
 // These ScoreKeepers don't get to draw.
 void ScoreKeeperShared::DrawPrimitives()
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::DrawPrimitives();
 }
 
 void ScoreKeeperShared::Update( float fDelta )
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::Update( fDelta );
 }
 
 void ScoreKeeperShared::OnNextSong( int iSongInCourseIndex, const Steps* pSteps, const NoteData* pNoteData )
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::OnNextSong( iSongInCourseIndex, pSteps, pNoteData );
 }
 
 void ScoreKeeperShared::HandleTapScore( const TapNote &tn )
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::HandleTapScore( tn );
 }
 
 void ScoreKeeperShared::HandleTapRowScore( const NoteData &nd, int iRow )
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::HandleTapRowScore( nd, iRow );
 }
 
 void ScoreKeeperShared::HandleHoldScore( const TapNote &tn )
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::HandleHoldScore( tn );
 }
 
 void ScoreKeeperShared::HandleHoldActiveSeconds( float fMusicSecondsHeld )
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::HandleHoldActiveSeconds( fMusicSecondsHeld );
 }
 
 void ScoreKeeperShared::HandleHoldCheckpointScore( const NoteData &nd, int iRow, int iNumHoldsHeldThisRow, int iNumHoldsMissedThisRow )
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::HandleHoldCheckpointScore( nd, iRow, iNumHoldsHeldThisRow,  iNumHoldsMissedThisRow );
 }
 
 void ScoreKeeperShared::HandleTapScoreNone()
 {
-	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->m_MasterPlayerNumber )
+	if( m_pPlayerState->m_PlayerNumber != GAMESTATE->GetMasterPlayerNumber() )
 		return;
 	ScoreKeeperNormal::HandleTapScoreNone();
 }

src/ScreenDebugOverlay.cpp

@@ -598,8 +598,8 @@ class DebugLineAutoplay : public IDebugLine
 	virtual bool IsEnabled() { return GamePreferences::m_AutoPlay.Get() != PC_HUMAN; }
 	virtual void DoAndLog( RString &sMessageOut )
 	{
-		ASSERT( GAMESTATE->m_MasterPlayerNumber != PLAYER_INVALID );
+		ASSERT( GAMESTATE->GetMasterPlayerNumber() != PLAYER_INVALID );
-		PlayerController pc = GAMESTATE->m_pPlayerState[GAMESTATE->m_MasterPlayerNumber]->m_PlayerController;
+		PlayerController pc = GAMESTATE->m_pPlayerState[GAMESTATE->GetMasterPlayerNumber()]->m_PlayerController;
 		bool bHoldingShift = 
 			INPUTFILTER->IsBeingPressed( DeviceInput(DEVICE_KEYBOARD, KEY_LSHIFT) ) || 
 			INPUTFILTER->IsBeingPressed( DeviceInput(DEVICE_KEYBOARD, KEY_RSHIFT) );
@@ -639,7 +639,7 @@ class DebugLineAssist : public IDebugLine
 	virtual bool IsEnabled() { return GAMESTATE->m_SongOptions.GetSong().m_bAssistClap || GAMESTATE->m_SongOptions.GetSong().m_bAssistMetronome; }
 	virtual void DoAndLog( RString &sMessageOut )
 	{
-		ASSERT( GAMESTATE->m_MasterPlayerNumber != PLAYER_INVALID );
+		ASSERT( GAMESTATE->GetMasterPlayerNumber() != PLAYER_INVALID );
 		bool bHoldingShift = INPUTFILTER->IsBeingPressed( DeviceInput(DEVICE_KEYBOARD, KEY_LSHIFT) );
 		bool b;
 		if( bHoldingShift )

src/ScreenEvaluation.cpp

@@ -99,7 +99,7 @@ void ScreenEvaluation::Init()
 		ss.m_Stage = Stage_1st;
 		enum_add( ss.m_Stage, rand()%3 );
 		ss.m_EarnedExtraStage = (EarnedExtraStage)(rand() % NUM_EarnedExtraStage);
-		GAMESTATE->m_MasterPlayerNumber = PLAYER_1;
+		GAMESTATE->SetMasterPlayerNumber(PLAYER_1);
 		GAMESTATE->m_pCurSong.Set( SONGMAN->GetRandomSong() );
 		ss.m_vpPlayedSongs.push_back( GAMESTATE->m_pCurSong );
 		ss.m_vpPossibleSongs.push_back( GAMESTATE->m_pCurSong );
@@ -635,7 +635,7 @@ void ScreenEvaluation::Init()
 			{
 			case PLAY_MODE_BATTLE:
 				{
-					bool bWon = GAMESTATE->GetStageResult(GAMESTATE->m_MasterPlayerNumber) == RESULT_WIN;
+					bool bWon = GAMESTATE->GetStageResult(GAMESTATE->GetMasterPlayerNumber()) == RESULT_WIN;
 					if( bWon )
 						SOUND->PlayOnceFromDir( ANNOUNCER->GetPathTo("evaluation win") );
 					else

src/ScreenGameplay.cpp

@@ -776,7 +776,7 @@ void ScreenGameplay::InitSongQueues()
 		ASSERT( pCourse );
 
 		m_apSongsQueue.clear();
-		PlayerNumber pnMaster = GAMESTATE->m_MasterPlayerNumber;
+		PlayerNumber pnMaster = GAMESTATE->GetMasterPlayerNumber();
 		Trail *pTrail = GAMESTATE->m_pCurTrail[pnMaster];
 		ASSERT( pTrail );
 		FOREACH_CONST( TrailEntry, pTrail->m_vEntries, e )
@@ -1227,7 +1227,7 @@ void ScreenGameplay::LoadNextSong()
 	FOREACH_EnabledPlayerInfo( m_vPlayerInfo, pi )
 	{
 		RageSoundReader *pPlayerSound = m_AutoKeysounds.GetPlayerSound(pi->m_pn);
-		if( pPlayerSound == NULL && pi->m_pn == GAMESTATE->m_MasterPlayerNumber )
+		if( pPlayerSound == NULL && pi->m_pn == GAMESTATE->GetMasterPlayerNumber() )
 			pPlayerSound = m_AutoKeysounds.GetSharedSound();
 		pi->m_SoundEffectControl.SetSoundReader( pPlayerSound );
 	}

src/ScreenGameplayShared.cpp

@@ -11,7 +11,7 @@ REGISTER_SCREEN_CLASS( ScreenGameplayShared );
 
 void ScreenGameplayShared::FillPlayerInfo( vector<PlayerInfo> &vPlayerInfoOut )
 {
-	const PlayerNumber master = GAMESTATE->m_MasterPlayerNumber;
+	const PlayerNumber master = GAMESTATE->GetMasterPlayerNumber();
 	const PlayerNumber other = (master == PLAYER_1? PLAYER_2:PLAYER_1);
 
 	/* The master player is where all of the real work takes place.  The other player exists
@@ -23,7 +23,7 @@ void ScreenGameplayShared::FillPlayerInfo( vector<PlayerInfo> &vPlayerInfoOut )
 
 PlayerInfo &ScreenGameplayShared::GetPlayerInfoForInput( const InputEventPlus& iep )
 {
-	return m_vPlayerInfo[GAMESTATE->m_MasterPlayerNumber];
+	return m_vPlayerInfo[GAMESTATE->GetMasterPlayerNumber()];
 }
 
 /*

src/ScreenHowToPlay.cpp

@@ -172,7 +172,7 @@ void ScreenHowToPlay::Init()
 
 		// Don't show judgment
 		PO_GROUP_ASSIGN( GAMESTATE->m_pPlayerState[PLAYER_1]->m_PlayerOptions, ModsLevel_Stage, m_fBlind, 1.0f );
-		GAMESTATE->m_MasterPlayerNumber = PLAYER_1;
+		GAMESTATE->SetMasterPlayerNumber(PLAYER_1);
 		GAMESTATE->m_bDemonstrationOrJukebox = true;
 	}
 

src/ScreenInstallOverlay.cpp

@@ -375,7 +375,7 @@ void ScreenInstallOverlay::Update( float fDeltaTime )
 			GAMEMAN->GetStylesForGame( GAMESTATE->m_pCurGame, vpStyle, false );
 			GAMESTATE->m_PlayMode.Set( PLAY_MODE_REGULAR );
 			GAMESTATE->m_bSideIsJoined[0] = true;
-			GAMESTATE->m_MasterPlayerNumber = PLAYER_1;
+			GAMESTATE->SetMasterPlayerNumber(PLAYER_1);
 			GAMESTATE->m_pCurStyle.Set( vpStyle[0] );
 			GAMESTATE->m_pCurSong.Set( pSong );
 			GAMESTATE->m_pPreferredSong = pSong;

src/ScreenJukebox.cpp

@@ -179,7 +179,7 @@ void ScreenJukebox::Init()
 
 //	ASSERT( GAMESTATE->m_pCurSong );
 
-	GAMESTATE->m_MasterPlayerNumber = PLAYER_1;
+	GAMESTATE->SetMasterPlayerNumber(PLAYER_1);
 
 	// choose some cool options
 	int Benchmark = 0;

src/ScreenMiniMenu.cpp

@@ -107,7 +107,7 @@ void ScreenMiniMenu::AfterChangeValueOrRow( PlayerNumber pn )
 			OptionRow &optrow = *m_pRows[i];
 			optrow.GetRowDef().m_vEnabledForPlayers.clear();
 			if( mr.pfnEnabled() )
-				optrow.GetRowDef().m_vEnabledForPlayers.insert( GAMESTATE->m_MasterPlayerNumber );
+				optrow.GetRowDef().m_vEnabledForPlayers.insert( GAMESTATE->GetMasterPlayerNumber() );
 		}
 		m_pRows[i]->UpdateEnabledDisabled();
 	}

src/ScreenNameEntry.cpp

@@ -118,7 +118,7 @@ ScreenNameEntry::ScreenNameEntry()
 	{
 		GAMESTATE->m_bSideIsJoined[PLAYER_1] = true;
 		GAMESTATE->m_bSideIsJoined[PLAYER_2] = true;
-		GAMESTATE->m_MasterPlayerNumber = PLAYER_1;
+		GAMESTATE->SetMasterPlayerNumber(PLAYER_1);
 		GAMESTATE->m_PlayMode.Set( PLAY_MODE_REGULAR );
 		GAMESTATE->SetCurrentStyle( GAMEMAN->GameAndStringToStyle( GAMEMAN->GetDefaultGame(),"versus") );
 		StageStats ss;

src/ScreenNameEntryTraditional.cpp

@@ -22,7 +22,7 @@ void ScreenNameEntryTraditional::Init()
 	{
 		GAMESTATE->m_bSideIsJoined[PLAYER_1] = true;
 		GAMESTATE->m_bSideIsJoined[PLAYER_2] = true;
-		GAMESTATE->m_MasterPlayerNumber = PLAYER_1;
+		GAMESTATE->SetMasterPlayerNumber(PLAYER_1);
 		GAMESTATE->m_PlayMode.Set( PLAY_MODE_REGULAR );
 		GAMESTATE->SetCurrentStyle( GAMEMAN->GameAndStringToStyle( GAMEMAN->GetDefaultGame(),"versus") );
 		for( int z = 0; z < 3; ++z )

src/ScreenOptions.cpp

@@ -531,7 +531,7 @@ void ScreenOptions::HandleScreenMessage( const ScreenMessage SM )
 			return; // already transitioning
 
 		// If the selected option sets a screen, honor it.
-		RString sThisScreen = GetNextScreenForFocusedItem( GAMESTATE->m_MasterPlayerNumber );
+		RString sThisScreen = GetNextScreenForFocusedItem( GAMESTATE->GetMasterPlayerNumber() );
 		if( sThisScreen != "" )
 			m_sNextScreen = sThisScreen;
 

src/ScreenOptionsCourseOverview.cpp

@@ -119,7 +119,7 @@ void ScreenOptionsCourseOverview::HandleScreenMessage( const ScreenMessage SM )
 	}
 	else if( SM == SM_GoToNextScreen )
 	{
-		int iRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+		int iRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 		switch( iRow )
 		{
 		case CourseOverviewRow_Play:
@@ -191,7 +191,7 @@ void ScreenOptionsCourseOverview::ProcessMenuStart( const InputEventPlus &input
 	if( IsTransitioning() )
 		return;
 
-	int iRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 	switch( iRow )
 	{
 	case CourseOverviewRow_Play:

src/ScreenOptionsEditCourse.cpp

@@ -488,7 +488,7 @@ void ScreenOptionsEditCourse::ProcessMenuStart( const InputEventPlus &input )
 	if( IsTransitioning() )
 		return;
 
-	int iRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 
 	unsigned iSongCount = GAMESTATE->m_pCurCourse->m_vEntries.size();
 

src/ScreenOptionsExportPackage.cpp

@@ -147,7 +147,7 @@ void ScreenOptionsExportPackage::ProcessMenuStart( const InputEventPlus &input )
 	if( IsTransitioning() )
 		return;
 
-	int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 	if( m_pRows[iCurRow]->GetRowType() == OptionRow::RowType_Exit )
 	{
 		ScreenOptions::ProcessMenuStart( input );

src/ScreenOptionsManageCourses.cpp

@@ -143,18 +143,18 @@ void ScreenOptionsManageCourses::BeginScreen()
 		if( iter != m_vpCourses.end() )
 		{
 			iIndex = iter - m_vpCourses.begin();
-			this->MoveRowAbsolute( GAMESTATE->m_MasterPlayerNumber, 1 + iIndex );
+			this->MoveRowAbsolute( GAMESTATE->GetMasterPlayerNumber(), 1 + iIndex );
 		}
 	}
 
-	AfterChangeRow( GAMESTATE->m_MasterPlayerNumber );
+	AfterChangeRow( GAMESTATE->GetMasterPlayerNumber() );
 }
 
 void ScreenOptionsManageCourses::HandleScreenMessage( const ScreenMessage SM )
 {
 	if( SM == SM_GoToNextScreen )
 	{
-		int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+		int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 
 		if( iCurRow == 0 )	// "create new"
 		{
@@ -218,7 +218,7 @@ void ScreenOptionsManageCourses::ProcessMenuStart( const InputEventPlus &input )
 	if( IsTransitioning() )
 		return;
 
-	int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 
 	if( iCurRow == 0 )	// "create new"
 	{
@@ -259,7 +259,7 @@ void ScreenOptionsManageCourses::ExportOptions( int iRow, const vector<PlayerNum
 
 Course *ScreenOptionsManageCourses::GetCourseWithFocus() const
 {
-	int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 	if( iCurRow == 0 )
 		return NULL;
 	else if( m_pRows[iCurRow]->GetRowType() == OptionRow::RowType_Exit )

src/ScreenOptionsManageEditSteps.cpp

@@ -125,7 +125,7 @@ void ScreenOptionsManageEditSteps::HandleScreenMessage( const ScreenMessage SM )
 {
 	if( SM == SM_GoToNextScreen )
 	{
-		int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+		int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 
 		if( iCurRow == 0 )	// "create new"
 		{
@@ -245,7 +245,7 @@ void ScreenOptionsManageEditSteps::ProcessMenuStart( const InputEventPlus &input
 	if( IsTransitioning() )
 		return;
 
-	int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 
 	if( iCurRow == 0 )	// "create new"
 	{
@@ -292,7 +292,7 @@ void ScreenOptionsManageEditSteps::ExportOptions( int iRow, const vector<PlayerN
 
 Steps *ScreenOptionsManageEditSteps::GetStepsWithFocus() const
 {
-	int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 	if( iCurRow == 0 )
 		return NULL;
 	else if( m_pRows[iCurRow]->GetRowType() == OptionRow::RowType_Exit )

src/ScreenOptionsManageProfiles.cpp

@@ -167,7 +167,7 @@ void ScreenOptionsManageProfiles::HandleScreenMessage( const ScreenMessage SM )
 {
 	if( SM == SM_GoToNextScreen )
 	{
-		int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+		int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 		OptionRow &row = *m_pRows[iCurRow];
 		if( row.GetRowType() == OptionRow::RowType_Exit )
 		{
@@ -337,7 +337,7 @@ void ScreenOptionsManageProfiles::ProcessMenuStart( const InputEventPlus &input
 	if( IsTransitioning() )
 		return;
 
-	int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 	OptionRow &row = *m_pRows[iCurRow];
 	
 	if( SHOW_CREATE_NEW && iCurRow == 0 )	// "create new"
@@ -405,7 +405,7 @@ void ScreenOptionsManageProfiles::ExportOptions( int iRow, const vector<PlayerNu
 
 int ScreenOptionsManageProfiles::GetLocalProfileIndexWithFocus() const
 {
-	int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 	OptionRow &row = *m_pRows[iCurRow];
 
 	if( SHOW_CREATE_NEW && iCurRow == 0 )	// "create new"

src/ScreenOptionsMemoryCard.cpp

@@ -111,7 +111,7 @@ void ScreenOptionsMemoryCard::AfterChangeRow( PlayerNumber pn )
 	}
 	else
 	{
-		int iRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+		int iRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 		m_textOsMountDir.SetText( m_CurrentUsbStorageDevices[iRow].sOsMountDir );
 	}
 }
@@ -124,7 +124,7 @@ void ScreenOptionsMemoryCard::HandleMessage( const Message &msg )
 		{
 			/* Remember the old mountpoint. */
 			const vector<UsbStorageDevice> &v = m_CurrentUsbStorageDevices;
-			int iRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+			int iRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 			RString sOldMountPoint;
 			if( iRow < int(v.size()) )
 			{
@@ -158,7 +158,7 @@ void ScreenOptionsMemoryCard::ExportOptions( int iRow, const vector<PlayerNumber
 	if( row.GetRowType() == OptionRow::RowType_Exit )
 		return;
 
-	PlayerNumber pn = GAMESTATE->m_MasterPlayerNumber;
+	PlayerNumber pn = GAMESTATE->GetMasterPlayerNumber();
 	if( m_iCurrentRow[pn] == iRow )
 	{
 		const vector<UsbStorageDevice> &v = m_CurrentUsbStorageDevices;
@@ -192,7 +192,7 @@ void ScreenOptionsMemoryCard::ProcessMenuStart( const InputEventPlus &input )
 	if( IsTransitioning() )
 		return;
 
-	int iCurRow = m_iCurrentRow[GAMESTATE->m_MasterPlayerNumber];
+	int iCurRow = m_iCurrentRow[GAMESTATE->GetMasterPlayerNumber()];
 
 	const vector<UsbStorageDevice> &v = m_CurrentUsbStorageDevices;
 	if( iCurRow < int(v.size()) )	// a card

src/ScreenSMOnlineLogin.cpp

@@ -155,7 +155,7 @@ void ScreenSMOnlineLogin::HandleScreenMessage(const ScreenMessage SM)
 	{
 		LOG->Trace("[ScreenSMOnlineLogin::HandleScreenMessage] GoToNextScreen");
 		vector<PlayerNumber> v;
-		v.push_back( GAMESTATE->m_MasterPlayerNumber );
+		v.push_back( GAMESTATE->GetMasterPlayerNumber() );
 		for( unsigned r=0; r<m_pRows.size(); r++ )
 			ExportOptions( r, v );
 

src/ScreenSelect.cpp

@@ -145,8 +145,8 @@ void ScreenSelect::HandleScreenMessage( const ScreenMessage SM )
 		 * same choice, and  if so, call ApplyToAll instead.
 		 * TODO: Think of a better way to handle this.
 		 */
-		ASSERT( GAMESTATE->m_MasterPlayerNumber != PlayerNumber_Invalid );
+		ASSERT( GAMESTATE->GetMasterPlayerNumber() != PlayerNumber_Invalid );
-		int iMastersIndex = this->GetSelectionIndex( GAMESTATE->m_MasterPlayerNumber );
+		int iMastersIndex = this->GetSelectionIndex( GAMESTATE->GetMasterPlayerNumber() );
 		bool bAllPlayersChoseTheSame = true;
 		FOREACH_HumanPlayer( p )
 		{

src/ScreenSelectMaster.cpp

@@ -294,7 +294,7 @@ void ScreenSelectMaster::HandleScreenMessage( const ScreenMessage SM )
 
 	if( SM == SM_PlayPostSwitchPage )
 	{
-		int iNewChoice = m_iChoice[ GAMESTATE->m_MasterPlayerNumber ];
+		int iNewChoice = m_iChoice[ GAMESTATE->GetMasterPlayerNumber() ];
 		Page newPage = GetPage( iNewChoice );
 
 		Message msg("PostSwitchPage");
@@ -516,7 +516,7 @@ void ScreenSelectMaster::MenuDown( const InputEventPlus &input )
 
 bool ScreenSelectMaster::ChangePage( int iNewChoice )
 {
-	Page oldPage = GetPage( m_iChoice[GAMESTATE->m_MasterPlayerNumber] );
+	Page oldPage = GetPage( m_iChoice[GAMESTATE->GetMasterPlayerNumber()] );
 	Page newPage = GetPage( iNewChoice );
 
 	// If anyone has already chosen, don't allow changing of pages
@@ -710,8 +710,8 @@ bool ScreenSelectMaster::ChangeSelection( PlayerNumber pn, MenuDir dir, int iNew
 
 PlayerNumber ScreenSelectMaster::GetSharedPlayer()
 {
-	if( GAMESTATE->m_MasterPlayerNumber != PLAYER_INVALID )
+	if( GAMESTATE->GetMasterPlayerNumber() != PLAYER_INVALID )
-		return GAMESTATE->m_MasterPlayerNumber;
+		return GAMESTATE->GetMasterPlayerNumber();
 
 	return PLAYER_1;
 }

src/ScreenSelectMusic.cpp

@@ -68,7 +68,7 @@ void ScreenSelectMusic::Init()
 		GAMESTATE->m_PlayMode.Set( PLAY_MODE_REGULAR );
 		GAMESTATE->SetCurrentStyle( GAMEMAN->GameAndStringToStyle(GAMEMAN->GetDefaultGame(),"versus") );
 		GAMESTATE->JoinPlayer( PLAYER_1 );
-		GAMESTATE->m_MasterPlayerNumber = PLAYER_1;
+		GAMESTATE->SetMasterPlayerNumber(PLAYER_1);
 	}
 
 	SAMPLE_MUSIC_DELAY_INIT.Load( m_sName, "SampleMusicDelayInit" );
@@ -1020,7 +1020,7 @@ void ScreenSelectMusic::HandleMessage( const Message &msg )
 
 		// TODO: Invalidate the CurSteps only if they are no longer playable.
 		// That way, after music change will clamp to the nearest in the StepsDisplayList.
-		GAMESTATE->m_pCurSteps[GAMESTATE->m_MasterPlayerNumber].SetWithoutBroadcast( NULL );
+		GAMESTATE->m_pCurSteps[GAMESTATE->GetMasterPlayerNumber()].SetWithoutBroadcast( NULL );
 		FOREACH_ENUM( PlayerNumber, p )
 			GAMESTATE->m_pCurSteps[p].SetWithoutBroadcast( NULL );
 
@@ -1392,7 +1392,7 @@ void ScreenSelectMusic::MenuStart( const InputEventPlus &input )
 			if( pStyle == NULL )
 			{
 				StepsType stCurrent;
-				PlayerNumber pn = GAMESTATE->m_MasterPlayerNumber;
+				PlayerNumber pn = GAMESTATE->GetMasterPlayerNumber();
 				if( GAMESTATE->IsCourseMode() )
 				{
 					ASSERT( GAMESTATE->m_pCurTrail[pn] );
@@ -1615,7 +1615,7 @@ void ScreenSelectMusic::SwitchToPreferredDifficulty()
 	if( GAMESTATE->DifficultiesLocked() )
 	{
 		FOREACH_HumanPlayer( p )
-			m_iSelection[p] = m_iSelection[GAMESTATE->m_MasterPlayerNumber];
+			m_iSelection[p] = m_iSelection[GAMESTATE->GetMasterPlayerNumber()];
 	}
 }
 

src/ScreenSongOptions.cpp

@@ -23,7 +23,7 @@ void ScreenSongOptions::Init()
 
 void ScreenSongOptions::ExportOptions( int iRow, const vector<PlayerNumber> &vpns )
 {
-	PlayerNumber pn = GAMESTATE->m_MasterPlayerNumber;
+	PlayerNumber pn = GAMESTATE->GetMasterPlayerNumber();
 	PlayerState *pPS = GAMESTATE->m_pPlayerState[pn];
 	const PlayerOptions::FailType ft = pPS->m_PlayerOptions.GetPreferred().m_FailType;
 

src/StatsManager.cpp

@@ -275,7 +275,7 @@ void StatsManager::CommitStatsToProfiles( const StageStats *pSS )
 			{
 				if( pSS->m_multiPlayer[mp].m_HighScore.IsEmpty() )
 					continue;
-				recent->AppendChild( MakeRecentScoreNode( *pSS, GAMESTATE->m_pCurTrail[GAMESTATE->m_MasterPlayerNumber], pSS->m_multiPlayer[mp], mp ) );
+				recent->AppendChild( MakeRecentScoreNode( *pSS, GAMESTATE->m_pCurTrail[GAMESTATE->GetMasterPlayerNumber()], pSS->m_multiPlayer[mp], mp ) );
 			}
 		}