Skip to content

Commit 16c89f3

Browse files
committed
refactor(network): use constexpr for NetPacket field type constants
1 parent 9fb4286 commit 16c89f3

File tree

2 files changed

+14
-12
lines changed

2 files changed

+14
-12
lines changed

Generals/Code/GameEngine/Source/GameNetwork/NetPacket.cpp

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,16 @@
3131
#include "GameNetwork/NetworkDefs.h"
3232
#include "GameNetwork/networkutil.h"
3333
#include "GameNetwork/GameMessageParser.h"
34+
#include <Utility/CppMacros.h>
3435

3536
// TheSuperHackers @refactor BobTista 10/06/2025 Extract magic character literals into named constants for improved readability
3637
namespace NetPacketFieldTypes {
37-
const char CommandType = 'T'; // NetCommandType field
38-
const char Relay = 'R'; // Relay field
39-
const char PlayerId = 'P'; // Player ID field
40-
const char CommandId = 'C'; // Command ID field
41-
const char Frame = 'F'; // Frame field
42-
const char Data = 'D'; // Data payload field
38+
constexpr char CommandType = 'T'; // NetCommandType field
39+
constexpr char Relay = 'R'; // Relay field
40+
constexpr char PlayerId = 'P'; // Player ID field
41+
constexpr char CommandId = 'C'; // Command ID field
42+
constexpr char Frame = 'F'; // Frame field
43+
constexpr char Data = 'D'; // Data payload field
4344
}
4445

4546
// This function assumes that all of the fields are either of default value or are

GeneralsMD/Code/GameEngine/Source/GameNetwork/NetPacket.cpp

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,16 @@
3131
#include "GameNetwork/NetworkDefs.h"
3232
#include "GameNetwork/networkutil.h"
3333
#include "GameNetwork/GameMessageParser.h"
34+
#include <Utility/CppMacros.h>
3435

3536
// TheSuperHackers @refactor BobTista 10/06/2025 Extract magic character literals into named constants for improved readability
3637
namespace NetPacketFieldTypes {
37-
const char CommandType = 'T'; // NetCommandType field
38-
const char Relay = 'R'; // Relay field
39-
const char PlayerId = 'P'; // Player ID field
40-
const char CommandId = 'C'; // Command ID field
41-
const char Frame = 'F'; // Frame field
42-
const char Data = 'D'; // Data payload field
38+
constexpr char CommandType = 'T'; // NetCommandType field
39+
constexpr char Relay = 'R'; // Relay field
40+
constexpr char PlayerId = 'P'; // Player ID field
41+
constexpr char CommandId = 'C'; // Command ID field
42+
constexpr char Frame = 'F'; // Frame field
43+
constexpr char Data = 'D'; // Data payload field
4344
}
4445

4546
// This function assumes that all of the fields are either of default value or are

0 commit comments

Comments
 (0)