cpp/openttd-patchpack/source Changeset - r26725:fb7e6e5eedb4

Changeset - r26725:fb7e6e5eedb4

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

Rubidium - 17 months ago 2023-01-03 18:02:36
rubidium@openttd.org

Codechange: silence some potentially uninitialized local variable errors

In these cases technically they are false positives, however dismissing the
alerts when the underlying code may make them true positives does not seem
like the safest solution.

2 files changed with 2 insertions and 2 deletions:

src/company_gui.cpp

src/script/api/script_rail.cpp

0 comments (0 inline, 0 general)

src/company_gui.cpp

➞

Show inline comments

@@ @@ -636,7 +636,7 @@ private: @@
 		const Company *c;
 		const Livery *livery, *default_livery = nullptr;
 		bool primary = widget == WID_SCL_PRI_COL_DROPDOWN;
 		byte default_col;
+		byte default_col = 0;
 		/* Disallow other company colours for the primary colour */
 		if (this->livery_class < LC_GROUP_RAIL && HasBit(this->sel, LS_DEFAULT) && primary) {

src/script/api/script_rail.cpp

➞

Show inline comments

@@ @@ -449,7 +449,7 @@ static bool IsValidSignalType(int signal @@
 	EnforcePrecondition(false, ::IsValidSignalType(signal));
 	Track track = INVALID_TRACK;
 	uint signal_cycles;
+	uint signal_cycles = 0;
 	int data_index = 2 + (::TileX(front) - ::TileX(tile)) + 2 * (::TileY(front) - ::TileY(tile));
 	for (int i = 0; i < NUM_TRACK_DIRECTIONS; i++) {

0 comments (0 inline, 0 general)