Skip to content

Commit

Permalink
Merge pull request #594 from lioncash/another-clamp-removal
Browse files Browse the repository at this point in the history
DolphinWX: Remove yet another clamp method
  • Loading branch information
delroth committed Jul 11, 2014
2 parents 59a7939 + dbdefd0 commit ef510a4
Showing 1 changed file with 3 additions and 9 deletions.
12 changes: 3 additions & 9 deletions Source/Core/DolphinWX/FrameAui.cpp
Expand Up @@ -33,6 +33,7 @@
#include "Common/Common.h"
#include "Common/FileUtil.h"
#include "Common/IniFile.h"
#include "Common/MathUtil.h"
#include "Common/StringUtil.h"
#include "Common/Logging/ConsoleListener.h"
#include "Core/ConfigManager.h"
Expand Down Expand Up @@ -733,13 +734,6 @@ void CFrame::OnSelectPerspective(wxCommandEvent& event)
DoLoadPerspective();
}

static int Limit(int i, int Low, int High)
{
if (i < Low) return Low;
if (i > High) return High;
return i;
}

void CFrame::SetPaneSize()
{
if (Perspectives.size() <= ActivePerspective)
Expand All @@ -764,8 +758,8 @@ void CFrame::SetPaneSize()
H = Perspectives[ActivePerspective].Height[j];

// Check limits
W = Limit(W, 5, 95);
H = Limit(H, 5, 95);
MathUtil::Clamp<u32>(&W, 5, 95);
MathUtil::Clamp<u32>(&H, 5, 95);

// Convert percentages to pixel lengths
W = (W * iClientX) / 100;
Expand Down

0 comments on commit ef510a4

Please sign in to comment.