Skip to content

Commit a4a57a2

Browse files
authored
refactor(client): Use constants for default display width and height values (#1465)
1 parent 0228421 commit a4a57a2

File tree

23 files changed

+63
-66
lines changed

23 files changed

+63
-66
lines changed

Core/GameEngine/Include/Common/GameDefines.h

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,3 +67,8 @@
6767
#define USE_OBSOLETE_GENERALS_CODE (1)
6868
#endif
6969
#endif
70+
71+
#define MIN_DISPLAY_BIT_DEPTH 16
72+
#define DEFAULT_DISPLAY_BIT_DEPTH 32
73+
#define DEFAULT_DISPLAY_WIDTH 800 // The standard resolution this game was designed for
74+
#define DEFAULT_DISPLAY_HEIGHT 600 // The standard resolution this game was designed for

Generals/Code/GameEngine/Source/Common/GlobalData.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -614,8 +614,8 @@ GlobalData::GlobalData()
614614
m_chipSetType = 0;
615615
m_headless = FALSE;
616616
m_windowed = 0;
617-
m_xResolution = 800;
618-
m_yResolution = 600;
617+
m_xResolution = DEFAULT_DISPLAY_WIDTH;
618+
m_yResolution = DEFAULT_DISPLAY_HEIGHT;
619619
m_maxShellScreens = 0;
620620
m_useCloudMap = FALSE;
621621
m_use3WayTerrainBlends = 1;

Generals/Code/GameEngine/Source/GameClient/GUI/ControlBar/ControlBarResizer.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -180,8 +180,8 @@ void ControlBarResizer::sizeWindowsAlt( void )
180180
{
181181
ResizerWindowList::iterator it = m_resizerWindowsList.begin();
182182
GameWindow *win = NULL;
183-
Real x = (Real)TheDisplay->getWidth() / 800;
184-
Real y = (Real)TheDisplay->getHeight() / 600;
183+
Real x = (Real)TheDisplay->getWidth() / DEFAULT_DISPLAY_WIDTH;
184+
Real y = (Real)TheDisplay->getHeight() / DEFAULT_DISPLAY_HEIGHT;
185185
while (it != m_resizerWindowsList.end())
186186
{
187187
ResizerWindow *rWin = *it;

Generals/Code/GameEngine/Source/GameClient/GUI/GUICallbacks/Menus/OptionsMenu.cpp

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -836,7 +836,7 @@ static void setDefaults( void )
836836
for( Int i = 0; i < numResolutions; ++i )
837837
{ Int xres,yres,bitDepth;
838838
TheDisplay->getDisplayModeDescription(i,&xres,&yres,&bitDepth);
839-
if (xres == 800 && yres == 600) //keep track of default mode in case we need it.
839+
if (xres == DEFAULT_DISPLAY_WIDTH && yres == DEFAULT_DISPLAY_HEIGHT) //keep track of default mode in case we need it.
840840
{ defaultResIndex=i;
841841
break;
842842
}
@@ -1648,12 +1648,15 @@ void OptionsMenuInit( WindowLayout *layout, void *userData )
16481648

16491649
// get resolution from saved preferences file
16501650
AsciiString selectedResolution = (*pref) ["Resolution"];
1651-
Int selectedXRes=800,selectedYRes=600;
1651+
Int selectedXRes=DEFAULT_DISPLAY_WIDTH;
1652+
Int selectedYRes=DEFAULT_DISPLAY_HEIGHT;
16521653
Int selectedResIndex=-1;
16531654
if (!selectedResolution.isEmpty())
16541655
{ //try to parse 2 integers out of string
16551656
if (sscanf(selectedResolution.str(),"%d%d", &selectedXRes, &selectedYRes) != 2)
1656-
{ selectedXRes=800; selectedYRes=600;
1657+
{
1658+
selectedXRes=DEFAULT_DISPLAY_WIDTH;
1659+
selectedYRes=DEFAULT_DISPLAY_HEIGHT;
16571660
}
16581661
}
16591662

Generals/Code/GameEngine/Source/GameClient/GUI/GUICallbacks/Menus/PopupPlayerInfo.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -456,8 +456,8 @@ void ResetBattleHonorInsertion(void)
456456
}
457457
void InsertBattleHonor(GameWindow *list, const Image *image, Bool enabled, Int itemData, Int& row, Int& column, UnicodeString text = UnicodeString::TheEmptyString)
458458
{
459-
Int width = MAX_BATTLE_HONOR_IMAGE_WIDTH * (TheDisplay->getWidth() / 800.0f);
460-
Int height = MAX_BATTLE_HONOR_IMAGE_HEIGHT * (TheDisplay->getHeight() / 600.0f);
459+
Int width = MAX_BATTLE_HONOR_IMAGE_WIDTH * (TheDisplay->getWidth() / (Real)DEFAULT_DISPLAY_WIDTH);
460+
Int height = MAX_BATTLE_HONOR_IMAGE_HEIGHT * (TheDisplay->getHeight() / (Real)DEFAULT_DISPLAY_HEIGHT);
461461

462462
static Int enabledColor = 0xFFFFFFFF;
463463
static Int disabledColor = GameMakeColor(80, 80, 80, 255);

Generals/Code/GameEngine/Source/GameClient/GUI/GUICallbacks/Menus/WOLWelcomeMenu.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -812,7 +812,7 @@ WindowMsgHandledType WOLWelcomeMenuSystem( GameWindow *window, UnsignedInt msg,
812812
else if (controlID == buttonQuickMatchID)
813813
{
814814
GameSpyMiscPreferences mPref;
815-
if ((TheDisplay->getWidth() != 800 || TheDisplay->getHeight() != 600) && mPref.getQuickMatchResLocked())
815+
if ((TheDisplay->getWidth() != DEFAULT_DISPLAY_WIDTH || TheDisplay->getHeight() != DEFAULT_DISPLAY_HEIGHT) && mPref.getQuickMatchResLocked())
816816
{
817817
GSMessageBoxOk(TheGameText->fetch("GUI:GSErrorTitle"), TheGameText->fetch("GUI:QuickMatch800x600"));
818818
}

Generals/Code/GameEngine/Source/GameClient/GlobalLanguage.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ void GlobalLanguage::parseFontFileName( INI *ini, void * instance, void *store,
187187

188188
Int GlobalLanguage::adjustFontSize(Int theFontSize)
189189
{
190-
Real adjustFactor = TheGlobalData->m_xResolution/800.0f;
190+
Real adjustFactor = TheGlobalData->m_xResolution / (Real)DEFAULT_DISPLAY_WIDTH;
191191
adjustFactor = 1.0f + (adjustFactor-1.0f) * m_resolutionFontSizeAdjustment;
192192
if (adjustFactor<1.0f) adjustFactor = 1.0f;
193193
if (adjustFactor>2.0f) adjustFactor = 2.0f;

Generals/Code/GameEngine/Source/GameClient/InGameUI.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4032,8 +4032,8 @@ void InGameUI::militarySubtitle( const AsciiString& label, Int duration )
40324032

40334033
// calculate where this screen position should be since the position being passed in is based off 8x6
40344034
Coord2D multiplyer;
4035-
multiplyer.x = TheDisplay->getWidth() / 800;
4036-
multiplyer.y = TheDisplay->getHeight() / 600;
4035+
multiplyer.x = TheDisplay->getWidth() / (Real)DEFAULT_DISPLAY_WIDTH;
4036+
multiplyer.y = TheDisplay->getHeight() / (Real)DEFAULT_DISPLAY_HEIGHT;
40374037

40384038
// lets bring out the data structure!
40394039
m_militarySubtitle = NEW MilitarySubtitleData;

Generals/Code/GameEngineDevice/Source/W3DDevice/GameClient/GUI/GUICallbacks/W3DMainMenu.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ static void advancePosition(GameWindow *window, const Image *image, UnsignedInt
108108
}
109109
static Int Width = size.x + image->getImageWidth();
110110

111-
static Int x = -800;
111+
static Int x = -DEFAULT_DISPLAY_WIDTH;
112112
static Int y = pos.y - (image->getImageHeight()/2);
113113

114114
static UnsignedInt m_startTime = timeGetTime();

Generals/Code/GameEngineDevice/Source/W3DDevice/GameClient/GUI/Gadget/W3DHorizontalSlider.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ void W3DGadgetHorizontalSliderImageDraw( GameWindow *window,
150150

151151
SliderData *s = (SliderData *)window->winGetUserData();
152152

153-
Real xMulti = INT_TO_REAL(TheDisplay->getWidth()) / 800;
153+
Real xMulti = INT_TO_REAL(TheDisplay->getWidth()) / DEFAULT_DISPLAY_WIDTH;
154154

155155
// figure out how many boxes we draw for this slider
156156
Int numBoxes = 0;
@@ -229,8 +229,8 @@ void W3DGadgetHorizontalSliderImageDrawB( GameWindow *window,
229229

230230
SliderData *s = (SliderData *)window->winGetUserData();
231231

232-
Real xMulti = INT_TO_REAL(TheDisplay->getWidth()) / 800;
233-
Real yMulti = INT_TO_REAL(TheDisplay->getHeight())/ 600;
232+
Real xMulti = INT_TO_REAL(TheDisplay->getWidth()) / DEFAULT_DISPLAY_WIDTH;
233+
Real yMulti = INT_TO_REAL(TheDisplay->getHeight()) / DEFAULT_DISPLAY_HEIGHT;
234234
// get image offset
235235
xOffset = instData->m_imageOffset.x;
236236
yOffset = instData->m_imageOffset.y;

0 commit comments

Comments
 (0)