Skip to content

Commit

Permalink
Remove CStereoscopicsManager instance
Browse files Browse the repository at this point in the history
The first step in killing globals is to identify if the service is
instantiated as a global or singleton. In this case, the service is a
singleton. Remove this instance.

Singletons usually have private constructors. Make all constructors public.
  • Loading branch information
garbear committed Mar 30, 2018
1 parent 5141e05 commit 3d2db95
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 11 deletions.
6 changes: 0 additions & 6 deletions xbmc/guilib/StereoscopicsManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -106,12 +106,6 @@ CStereoscopicsManager::CStereoscopicsManager(void)

CStereoscopicsManager::~CStereoscopicsManager(void) = default;

CStereoscopicsManager& CStereoscopicsManager::GetInstance()
{
static CStereoscopicsManager sStereoscopicsManager;
return sStereoscopicsManager;
}

void CStereoscopicsManager::Initialize(void)
{
// turn off stereo mode on XBMC startup
Expand Down
5 changes: 0 additions & 5 deletions xbmc/guilib/StereoscopicsManager.h
Original file line number Diff line number Diff line change
Expand Up @@ -48,11 +48,6 @@ class CStereoscopicsManager : public ISettingCallback,
CStereoscopicsManager(void);
~CStereoscopicsManager(void) override;

/*!
* @return static instance of CStereoscopicsManager
*/
static CStereoscopicsManager& GetInstance();

void Initialize(void);
RENDER_STEREO_MODE GetStereoMode(void);
void SetStereoModeByUser(const RENDER_STEREO_MODE &mode);
Expand Down

0 comments on commit 3d2db95

Please # to comment.