Skip to content

Commit 7755c64

Browse files
committed
Replace global access to Data Cache Core with reference
Here, we do the same thing as CSettings with CDataCacheCore. Notice that we place the new reference on the next line following CSettings so we can scale to handle lots of dependencies with a clean git history.
1 parent cbcf525 commit 7755c64

File tree

3 files changed

+11
-5
lines changed

3 files changed

+11
-5
lines changed

xbmc/ServiceManager.cpp

+2-1
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,8 @@ bool CServiceManager::StartAudioEngine()
230230
// stage 3 is called after successful initialization of WindowManager
231231
bool CServiceManager::InitStageThree()
232232
{
233-
m_stereoscopicsManager.reset(new CStereoscopicsManager(*m_settings));
233+
m_stereoscopicsManager.reset(new CStereoscopicsManager(*m_settings,
234+
*m_dataCacheCore));
234235
m_stereoscopicsManager->Initialize();
235236

236237
// Peripherals depends on strings being loaded before stage 3

xbmc/guilib/StereoscopicsManager.cpp

+5-3
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,10 @@ static const struct StereoModeMap StringToGuiModeMap[] =
9898
};
9999

100100

101-
CStereoscopicsManager::CStereoscopicsManager(CSettings &settings) :
102-
m_settings(settings)
101+
CStereoscopicsManager::CStereoscopicsManager(CSettings &settings,
102+
CDataCacheCore &dataCacheCore) :
103+
m_settings(settings),
104+
m_dataCacheCore(dataCacheCore)
103105
{
104106
m_stereoModeSetByUser = RENDER_STEREO_MODE_UNDEFINED;
105107
m_lastStereoModeSetByUser = RENDER_STEREO_MODE_UNDEFINED;
@@ -495,7 +497,7 @@ std::string CStereoscopicsManager::GetVideoStereoMode()
495497
{
496498
std::string playerMode;
497499
if (g_application.GetAppPlayer().IsPlaying())
498-
playerMode = CServiceBroker::GetDataCacheCore().GetVideoStereoMode();
500+
playerMode = m_dataCacheCore.GetVideoStereoMode();
499501
return playerMode;
500502
}
501503

xbmc/guilib/StereoscopicsManager.h

+4-1
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
#include "rendering/RenderSystem.h"
3232

3333
class CAction;
34+
class CDataCacheCore;
3435
class CSettings;
3536

3637
enum STEREOSCOPIC_PLAYBACK_MODE
@@ -46,7 +47,8 @@ class CStereoscopicsManager : public ISettingCallback,
4647
public IMsgTargetCallback
4748
{
4849
public:
49-
CStereoscopicsManager(CSettings &settings);
50+
CStereoscopicsManager(CSettings &settings,
51+
CDataCacheCore &dataCacheCore);
5052
~CStereoscopicsManager(void) override;
5153

5254
void Initialize(void);
@@ -93,6 +95,7 @@ class CStereoscopicsManager : public ISettingCallback,
9395

9496
// Construction parameters
9597
CSettings &m_settings;
98+
CDataCacheCore &m_dataCacheCore;
9699

97100
// Stereoscopic parameters
98101
RENDER_STEREO_MODE m_stereoModeSetByUser;

0 commit comments

Comments
 (0)