Skip to content

Commit 35bee47

Browse files
committed
[GEN][ZH] Cleans up inconsistencies ()
1 parent bbf0830 commit 35bee47

File tree

368 files changed

+8098
-8251
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

368 files changed

+8098
-8251
lines changed

Generals/Code/GameEngine/Include/Common/ArchiveFileSystem.h

-1
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,6 @@ class ArchiveFileSystem : public SubsystemInterface
172172

173173
// Unprotected this for copy-protection routines
174174
AsciiString getArchiveFilenameForFile(const AsciiString& filename) const;
175-
176175
void loadMods( void );
177176

178177
protected:

Generals/Code/GameEngine/Include/Common/BuildAssistant.h

+6-6
Original file line numberDiff line numberDiff line change
@@ -112,12 +112,12 @@ class BuildAssistant : public SubsystemInterface
112112

113113
enum LocalLegalToBuildOptions
114114
{
115-
TERRAIN_RESTRICTIONS = 0x00000001, ///< Check for basic terrain restrictions
116-
CLEAR_PATH = 0x00000002, ///< Must be able to path find to location
117-
NO_OBJECT_OVERLAP = 0X00000004, ///< Can't overlap enemy objects, or locally controled objects that can't move out of the way
118-
USE_QUICK_PATHFIND = 0x00000008, ///< Use the quick pathfind method for CLEAR_PATH
119-
SHROUD_REVEALED = 0x00000010, ///< Check to make sure the shroud is revealed
120-
NO_ENEMY_OBJECT_OVERLAP=0x00000020, ///< Can't overlap enemy objects only.
115+
TERRAIN_RESTRICTIONS = 0x00000001, ///< Check for basic terrain restrictions
116+
CLEAR_PATH = 0x00000002, ///< Must be able to path find to location
117+
NO_OBJECT_OVERLAP = 0X00000004, ///< Can't overlap enemy objects, or locally controled objects that can't move out of the way
118+
USE_QUICK_PATHFIND = 0x00000008, ///< Use the quick pathfind method for CLEAR_PATH
119+
SHROUD_REVEALED = 0x00000010, ///< Check to make sure the shroud is revealed
120+
NO_ENEMY_OBJECT_OVERLAP = 0x00000020, ///< Can't overlap enemy objects only.
121121
};
122122

123123
public:

Generals/Code/GameEngine/Include/Common/FileSystem.h

-2
Original file line numberDiff line numberDiff line change
@@ -138,8 +138,6 @@ class FileSystem : public SubsystemInterface
138138
void loadMusicFilesFromCD();
139139
void unloadMusicFilesFromCD();
140140
protected:
141-
142-
143141
};
144142

145143
extern FileSystem* TheFileSystem;

Generals/Code/GameEngine/Include/Common/GameMemory.h

+2-5
Original file line numberDiff line numberDiff line change
@@ -730,7 +730,6 @@ private: \
730730
public: /* include this line at the end to reset visibility to 'public' */
731731

732732

733-
734733
// ----------------------------------------------------------------------------
735734
/**
736735
This class is provided as a simple and safe way to integrate C++ object allocation
@@ -768,7 +767,6 @@ class MemoryPoolObject
768767
};
769768

770769

771-
772770
// INLINING ///////////////////////////////////////////////////////////////////
773771

774772
// ----------------------------------------------------------------------------
@@ -890,10 +888,9 @@ class STLSpecialAlloc
890888
static void deallocate(void* __p, size_t);
891889
};
892890

893-
#endif //DISABLE_GAMEMEMORY
891+
#endif // DISABLE_GAMEMEMORY
894892

895893

896-
// ----------------------------------------------------------------------------
897894
/**
898895
A simple utility class to ensure exception safety; this holds a MemoryPoolObject
899896
and deletes it in its destructor. Especially useful for iterators!
@@ -910,7 +907,6 @@ class MemoryPoolObjectHolder
910907
};
911908

912909

913-
// ----------------------------------------------------------------------------
914910
/**
915911
Sometimes you want to make a class's destructor protected so that it can only
916912
be destroyed under special circumstances. MemoryPoolObject short-circuits this
@@ -924,4 +920,5 @@ ARGVIS: void deleteInstance() { MemoryPoolObject::deleteInstance(); } public:
924920

925921
#define EMPTY_DTOR(CLASS) inline CLASS::~CLASS() { }
926922

923+
927924
#endif // _GAME_MEMORY_H_

Generals/Code/GameEngine/Include/Common/GlobalData.h

+1-2
Original file line numberDiff line numberDiff line change
@@ -412,7 +412,7 @@ class GlobalData : public SubsystemInterface
412412
UnsignedInt m_unlookPersistDuration; ///< How long after unlook until the sighting info executes the undo
413413

414414
Bool m_shouldUpdateTGAToDDS; ///< Should we attempt to update old TGAs to DDS stuff on loadup?
415-
415+
416416
UnsignedInt m_doubleClickTimeMS; ///< What is the maximum amount of time that can seperate two clicks in order
417417
///< for us to generate a double click message?
418418

@@ -486,7 +486,6 @@ class GlobalData : public SubsystemInterface
486486

487487
Bool m_isBreakableMovie; ///< if we enter a breakable movie, set this flag
488488
Bool m_breakTheMovie; ///< The user has hit escape!
489-
490489
AsciiString m_modDir;
491490
AsciiString m_modBIG;
492491

Generals/Code/GameEngine/Include/Common/KindOf.h

-1
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,6 @@ enum KindOfType
144144
KINDOF_DONT_AUTO_CRUSH_INFANTRY, ///< These units don't try to crush the infantry if ai.
145145

146146
KINDOF_COUNT // total number of kindofs
147-
148147
};
149148

150149
typedef BitFlags<KINDOF_COUNT> KindOfMaskType;

Generals/Code/GameEngine/Include/Common/MiscAudio.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -73,4 +73,4 @@ struct MiscAudio
7373
};
7474

7575

76-
#endif /* _MISCAUDIO_H_ */
76+
#endif /* _MISCAUDIO_H_ */

Generals/Code/GameEngine/Include/Common/Money.h

-1
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,6 @@ class Money : public Snapshot
8383
void setPlayerIndex(Int ndx) { m_playerIndex = ndx; }
8484

8585
protected:
86-
8786
// snapshot methods
8887
virtual void crc( Xfer *xfer );
8988
virtual void xfer( Xfer *xfer );

Generals/Code/GameEngine/Include/Common/PlayerTemplate.h

-2
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,6 @@ class PlayerTemplate
126126
Int getSpecialPowerShortcutButtonCount( void ) const {return m_specialPowerShortcutButtonCount; }
127127

128128
AsciiString getLoadScreenMusic( void ) const {return m_loadScreenMusic; }
129-
130129

131130
static const FieldParse* getFieldParse();
132131

@@ -200,7 +199,6 @@ class PlayerTemplateStore : public SubsystemInterface
200199
const PlayerTemplate* getNthPlayerTemplate(Int i) const;
201200
const PlayerTemplate* findPlayerTemplate(NameKeyType namekey) const;
202201
inline Int getPlayerTemplateCount() const { return m_playerTemplates.size(); }
203-
204202

205203
// This function will fill outStringList with all the sides found in all the templates
206204
void getAllSideStrings(AsciiStringList *outStringList);

Generals/Code/GameEngine/Include/Common/StackDump.h

+1-3
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,7 @@
2929

3030
#ifndef IG_DEGBUG_STACKTRACE
3131
#define IG_DEBUG_STACKTRACE 1
32-
#endif
33-
32+
#endif // Unsure about this one -ML 3/25/03
3433
#if defined(_DEBUG) || defined(_INTERNAL) || defined(IG_DEBUG_STACKTRACE)
3534

3635
// Writes a stackdump (provide a callback : gets called per line)
@@ -71,5 +70,4 @@ __inline void DumpExceptionInfo( unsigned int u, EXCEPTION_POINTERS* e_info ) {}
7170

7271
extern AsciiString g_LastErrorDump;
7372

74-
7573
#endif // __STACKDUMP_H_

Generals/Code/GameEngine/Include/Common/ThingTemplate.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,7 @@ class ThingTemplate : public Overridable
510510
const FXList* getPerUnitFX(const AsciiString& fxName) const;
511511

512512
UnsignedInt getThreatValue() const { return m_threatValue; }
513-
UnsignedInt getMaxSimultaneousOfType() const { return m_maxSimultaneousOfType; }
513+
UnsignedInt getMaxSimultaneousOfType() const { return m_maxSimultaneousOfType; }
514514

515515
void validate();
516516

Generals/Code/GameEngine/Include/GameClient/CommandXlat.h

-1
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,6 @@ enum FilterModes
105105
FM_VIEW_MB_END_PAN_ALPHA, // Moton blur on screen pan (for camera tracks object mode)
106106

107107

108-
109108
// NOTE: This has to be the last entry in this enum.
110109
// Add new entries before this one. jba.
111110
FM_VIEW_MB_PAN_ALPHA, // Moton blur on screen pan (for camera tracks object mode)

Generals/Code/GameEngine/Include/GameClient/ControlBar.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ enum GUICommandType
186186
GUI_COMMAND_FIRE_WEAPON, ///< fire a weapon
187187
GUI_COMMAND_SPECIAL_POWER, ///< do a special power
188188
GUI_COMMAND_PURCHASE_SCIENCE, ///< purchase science
189-
GUI_COMMAND_HACK_INTERNET, ///< Hey author, write me!
189+
GUI_COMMAND_HACK_INTERNET, ///< gain income from the ether (by hacking the internet)
190190
GUI_COMMAND_TOGGLE_OVERCHARGE, ///< Overcharge command for power plants
191191
#ifdef ALLOW_SURRENDER
192192
GUI_COMMAND_POW_RETURN_TO_PRISON, ///< POW Truck, return to prison

Generals/Code/GameEngine/Include/GameClient/Eva.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -176,4 +176,4 @@ class Eva : public SubsystemInterface
176176

177177
extern Eva *TheEva;
178178

179-
#endif /* __EVA_H__ */
179+
#endif /* __EVA_H__ */

Generals/Code/GameEngine/Include/GameClient/GameClient.h

-1
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,6 @@ class GameClient : public SubsystemInterface,
183183
virtual TerrainVisual *createTerrainVisual( void ) = 0; ///< Factory for TerrainVisual classes. Called during init to instance TheTerrainVisual
184184
virtual Keyboard *createKeyboard( void ) = 0; ///< factory for the keyboard
185185
virtual Mouse *createMouse( void ) = 0; ///< factory for the mouse
186-
187186
virtual void setFrameRate(Real msecsPerFrame) = 0;
188187

189188
// ----------------------------------------------------------------------------------------------

Generals/Code/GameEngine/Include/GameClient/InGameUI.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -37,14 +37,14 @@
3737
#include "Common/GameType.h"
3838
#include "Common/MessageStream.h" // for GameMessageTranslator
3939
#include "Common/SpecialPowerType.h"
40+
#include "Common/Snapshot.h"
4041
#include "Common/STLTypedefs.h"
4142
#include "Common/SubsystemInterface.h"
4243
#include "Common/UnicodeString.h"
4344
#include "GameClient/DisplayString.h"
4445
#include "GameClient/Mouse.h"
4546
#include "GameClient/RadiusDecal.h"
4647
#include "GameClient/View.h"
47-
#include "Common/Snapshot.h"
4848

4949
// FORWARD DECLARATIONS ///////////////////////////////////////////////////////////////////////////
5050
class Drawable;
@@ -118,7 +118,7 @@ static const char *TheRadiusCursorNames[] =
118118
"CARPETBOMB",
119119
"DAISYCUTTER",
120120
"PARADROP",
121-
"SPYSATELLITE",
121+
"SPYSATELLITE",
122122

123123
"NUCLEARMISSILE",
124124
"EMPPULSE",

Generals/Code/GameEngine/Include/GameClient/Mouse.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,7 @@ class Mouse : public SubsystemInterface
281281
void mouseNotifyResolutionChange(void);
282282

283283
Bool isClick(const ICoord2D *anchor, const ICoord2D *dest, UnsignedInt previousMouseClick, UnsignedInt currentMouseClick);
284-
284+
285285
AsciiString m_tooltipFontName; ///< tooltip font
286286
Int m_tooltipFontSize; ///< tooltip font
287287
Bool m_tooltipFontIsBold; ///< tooltip font

Generals/Code/GameEngine/Include/GameClient/TerrainVisual.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ class TerrainVisual : public Snapshot,
156156
// Modify height.
157157
//
158158
virtual void setRawMapHeight(const ICoord2D *gridPos, Int height)=0;
159-
159+
160160
/// Replace the skybox texture
161161
virtual void replaceSkyboxTextures(const AsciiString *oldTexName[NumSkyboxTextures], const AsciiString *newTexName[NumSkyboxTextures])=0;
162162

Generals/Code/GameEngine/Include/GameClient/View.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,8 @@ class View : public Snapshot
144144
virtual void setFadeParameters(Int fadeFrames, Int direction) { };
145145
virtual void set3DWireFrameMode(Bool enable) { };
146146

147-
virtual void resetCamera(const Coord3D *location, Int frames) {}; ///< Move camera to location, and reset to default angle & zoom.
148-
virtual void rotateCamera(Real rotations, Int frames) {}; ///< Rotate camera about current viewpoint.
147+
virtual void resetCamera(const Coord3D *location, Int frames) {}; ///< Move camera to location, and reset to default angle & zoom.
148+
virtual void rotateCamera(Real rotations, Int frames) {}; ///< Rotate camera about current viewpoint.
149149
virtual void rotateCameraTowardObject(ObjectID id, Int milliseconds, Int holdMilliseconds) {}; ///< Rotate camera to face an object, and hold on it
150150
virtual void rotateCameraTowardPosition(const Coord3D *pLoc, Int milliseconds) {}; ///< Rotate camera to face a location.
151151
virtual Bool isTimeFrozen(void){ return false;} ///< Freezes time during the next camera movement.

Generals/Code/GameEngine/Include/GameLogic/AI.h

+2-1
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,8 @@ class Weapon;
307307
// TheSuperHackers @compile xezon 22/03/2025 Renames AI_PASSIVE to not conflict with macro in ws2def.h
308308

309309
// Note - written out in save/load xfer and .map files, don't change these numbers.
310-
enum AttitudeType {
310+
enum AttitudeType
311+
{
311312
ATTITUDE_SLEEP = -2,
312313
ATTITUDE_PASSIVE=-1,
313314
ATTITUDE_NORMAL=0,

Generals/Code/GameEngine/Include/GameLogic/Damage.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@
3636
#include "Common/GameType.h"
3737
#include "Common/Snapshot.h"
3838

39+
3940
// FORWARD REFERENCES /////////////////////////////////////////////////////////////////////////////
4041
class Object;
4142
class INI;
@@ -215,7 +216,6 @@ static const char *TheDeathNames[] =
215216
"DETONATED",
216217
"SPLATTED",
217218
"POISONED_BETA",
218-
219219
"EXTRA_2",
220220
"EXTRA_3",
221221
"EXTRA_4",

Generals/Code/GameEngine/Include/GameLogic/GameLogic.h

+2
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ class GhostObjectManager;
6868
class CommandButton;
6969
enum BuildableStatus;
7070

71+
7172
typedef const CommandButton* ConstCommandButtonPtr;
7273

7374
// What kind of game we're in.
@@ -88,6 +89,7 @@ enum
8889
CRC_RECALC
8990
};
9091

92+
9193
/// Function pointers for use by GameLogic callback functions.
9294
typedef void (*GameLogicFuncPtr)( Object *obj, void *userData );
9395
typedef std::hash_map<ObjectID, Object *, rts::hash<ObjectID>, rts::equal_to<ObjectID> > ObjectPtrHash;

Generals/Code/GameEngine/Include/GameLogic/Module/BehaviorModule.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -209,8 +209,8 @@ class ParkingPlaceBehaviorInterface
209209
Coord3D parkingSpace;
210210
Real parkingOrientation;
211211
Coord3D runwayPrep;
212-
Coord3D runwayStart;
213-
Coord3D runwayEnd;
212+
Coord3D runwayStart;
213+
Coord3D runwayEnd;
214214
Coord3D runwayApproach;
215215
Coord3D hangarInternal;
216216
Real hangarInternalOrient;

Generals/Code/GameEngine/Include/GameLogic/Module/ConvertToCarBombCrateCollide.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
// //
2323
////////////////////////////////////////////////////////////////////////////////
2424

25-
// FILE: VeterancyCrateCollide.h /////////////////////////////////////////////////////////////////////////
25+
// FILE: ConvertToCarBombCrateCollide.h /////////////////////////////////////////////////////////////////////////
2626
// Author: Kris Morness, April 2002
2727
// Desc: A crate (actually a terrorist - mobile crate) that converts a car into a carbomb, activating
2828
// it's weapon and then activating it's AI.

Generals/Code/GameEngine/Include/GameLogic/Module/EMPUpdate.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ class EMPUpdateModuleData : public UpdateModuleData
4949
RGBColor m_startColor;
5050
RGBColor m_endColor;
5151
const ParticleSystemTemplate *m_disableFXParticleSystem;
52-
Real m_sparksPerCubicFoot; //<just like it sounds
52+
Real m_sparksPerCubicFoot; //<just like it sounds
5353

5454
EMPUpdateModuleData()
5555
{

Generals/Code/GameEngine/Include/GameLogic/Module/FXListDie.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ class FXList;
4646
class FXListDieModuleData : public DieModuleData
4747
{
4848
public:
49-
const FXList *m_defaultDeathFX; ///< default fx to make
50-
Bool m_orientToObject;
49+
const FXList *m_defaultDeathFX; ///< default fx to make
50+
Bool m_orientToObject;
5151

5252
FXListDieModuleData()
5353
{

Generals/Code/GameEngine/Include/GameLogic/Module/FireWeaponWhenDeadBehavior.h

+1
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
#define __FireWeaponWhenDeadBehavior_H_
3434

3535
// INCLUDES ///////////////////////////////////////////////////////////////////////////////////////
36+
3637
#include "GameLogic/Module/BehaviorModule.h"
3738
#include "GameLogic/Module/DieModule.h"
3839
#include "GameLogic/Module/UpgradeModule.h"

Generals/Code/GameEngine/Include/GameLogic/Module/GarrisonContain.h

+1
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@ class GarrisonContainModuleData : public OpenContainModuleData
5454
Real m_framesForFullHeal;
5555
Bool m_mobileGarrison;
5656
Bool m_immuneToClearBuildingAttacks;
57+
5758
InitialRoster m_initialRoster;
5859

5960
GarrisonContainModuleData( void );

Generals/Code/GameEngine/Include/GameLogic/Module/GenerateMinefieldBehavior.h

+4-5
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@
3737
#include "GameLogic/Module/DieModule.h"
3838
#include "GameLogic/Module/UpgradeModule.h"
3939

40-
4140
//-------------------------------------------------------------------------------------------------
4241
class GenerateMinefieldBehaviorModuleData : public BehaviorModuleData
4342
{
@@ -110,10 +109,10 @@ class GenerateMinefieldBehavior : public BehaviorModule,
110109

111110

112111
private:
113-
Coord3D m_target;
114-
Bool m_hasTarget;
115-
Bool m_generated;
116-
112+
Coord3D m_target;
113+
Bool m_hasTarget;
114+
Bool m_generated;
115+
117116
const Coord3D* getMinefieldTarget() const;
118117
void placeMines();
119118
void placeMinesInFootprint(const GeometryInfo& geom, const ThingTemplate* mineTemplate);

Generals/Code/GameEngine/Include/GameLogic/Module/OCLUpdate.h

+6-6
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,10 @@ class ObjectCreationList;
4141
class OCLUpdateModuleData : public UpdateModuleData
4242
{
4343
public:
44-
const ObjectCreationList *m_ocl;
45-
UnsignedInt m_minDelay;
46-
UnsignedInt m_maxDelay;
47-
Bool m_isCreateAtEdge; ///< Otherwise, it is created on top of myself
44+
const ObjectCreationList * m_ocl;
45+
UnsignedInt m_minDelay;
46+
UnsignedInt m_maxDelay;
47+
Bool m_isCreateAtEdge; ///< Otherwise, it is created on top of myself
4848

4949
OCLUpdateModuleData();
5050

@@ -74,8 +74,8 @@ class OCLUpdate : public UpdateModule
7474

7575
protected:
7676

77-
UnsignedInt m_nextCreationFrame;
78-
UnsignedInt m_timerStartedFrame;
77+
UnsignedInt m_nextCreationFrame;
78+
UnsignedInt m_timerStartedFrame;
7979

8080
Bool shouldCreate();
8181
void setNextCreationFrame();

Generals/Code/GameEngine/Include/GameLogic/Module/PhysicsUpdate.h

+1
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ class PhysicsBehaviorModuleData : public UpdateModuleData
6262
Real m_minFallSpeedForDamage;
6363
Real m_fallHeightDamageFactor;
6464
Real m_pitchRollYawFactor;
65+
6566
const WeaponTemplate* m_vehicleCrashesIntoBuildingWeaponTemplate;
6667
const WeaponTemplate* m_vehicleCrashesIntoNonBuildingWeaponTemplate;
6768

0 commit comments

Comments
 (0)