Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Add an OSD message to remind the user if Shader Debugging is enabled
Fixes issue 6497.
  • Loading branch information
delroth committed Aug 17, 2013
1 parent 07d729d commit d6fe9c6
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions Source/Core/VideoCommon/Src/VideoConfig.cpp
Expand Up @@ -11,6 +11,7 @@
#include "FileUtil.h"
#include "Core.h"
#include "Movie.h"
#include "OnScreenDisplay.h"

VideoConfig g_Config;
VideoConfig g_ActiveConfig;
Expand Down Expand Up @@ -104,6 +105,18 @@ void VideoConfig::Load(const char *ini_file)
bool bTmp;
iniFile.Get("Interface", "UsePanicHandlers", &bTmp, true);
SetEnableAlert(bTmp);

// Shader Debugging causes a huge slowdown and it's easy to forget about it
// since it's not exposed in the settings dialog. It's only used by
// developers, so displaying an obnoxious message avoids some confusion and
// is not too annoying/confusing for users.
//
// XXX(delroth): This is kind of a bad place to put this, but the current
// VideoCommon is a mess and we don't have a central initialization
// function to do these kind of checks. Instead, the init code is
// triplicated for each video backend.
if (bEnableShaderDebugging)
OSD::AddMessage("Warning: Shader Debugging is enabled, performance will suffer heavily", 15000);
}

void VideoConfig::GameIniLoad(const char *ini_file)
Expand Down

0 comments on commit d6fe9c6

Please sign in to comment.