cpp/openttd-patchpack/source Changeset - r26038:ced62dd5d288

Changeset - r26038:ced62dd5d288

Parent rev.

Child rev.

[Not reviewed]

master

0 5 0

frosch - 3 years ago 2021-10-22 19:46:55
frosch@openttd.org

Codechange: remove redundant 'info_view' parameter, it can be derived from 'image_type'.

5 files changed with 8 insertions and 7 deletions:

src/aircraft_cmd.cpp

src/aircraft_gui.cpp

src/newgrf_engine.cpp

src/newgrf_engine.h

src/vehicle_gui.cpp

0 comments (0 inline, 0 general)

src/aircraft_cmd.cpp

➞

Show inline comments

@@ @@ -188,7 +188,7 @@ void GetRotorImage(const Aircraft *v, En @@
 	const Aircraft *w = v->Next()->Next();
 	if (is_custom_sprite(v->spritenum)) {
-		GetCustomRotorSprite(v, false, image_type, result);
 		GetCustomRotorSprite(v, image_type, result);
 		if (result->IsValid()) return;
+	}

src/aircraft_gui.cpp

➞

Show inline comments

@@ @@ -100,7 +100,7 @@ void DrawAircraftImage(const Vehicle *v, @@
 	if (helicopter) {
 		const Aircraft *a = Aircraft::From(v);
 		VehicleSpriteSeq rotor_seq;
-		GetCustomRotorSprite(a, true, image_type, &rotor_seq);
 		GetCustomRotorSprite(a, image_type, &rotor_seq);
 		if (!rotor_seq.IsValid()) rotor_seq.Set(SPR_ROTOR_STOPPED);
 		heli_offs = ScaleGUITrad(5);
 		rotor_seq.Draw(x, y + y_offs - heli_offs, PAL_NONE, false);

src/newgrf_engine.cpp

➞

Show inline comments

@@ @@ -1103,7 +1103,7 @@ void GetCustomEngineSprite(EngineID engi @@
+}
-void GetRotorOverrideSprite(EngineID engine, const struct Aircraft *v, bool info_view, EngineImageType image_type, VehicleSpriteSeq *result)
 void GetRotorOverrideSprite(EngineID engine, const struct Aircraft *v, EngineImageType image_type, VehicleSpriteSeq *result)
+{
 	const Engine *e = Engine::Get(engine);
@@ @@ -1111,6 +1111,7 @@ void GetRotorOverrideSprite(EngineID eng @@
 	assert(e->type == VEH_AIRCRAFT);
 	assert(!(e->u.air.subtype & AIR_CTOL));
 	bool info_view = image_type != EIT_ON_MAP;
 	VehicleResolverObject object(engine, v, VehicleResolverObject::WO_SELF, info_view, CBID_NO_CALLBACK);
 	result->Clear();
 	uint rotor_pos = v == nullptr || info_view ? 0 : v->Next()->Next()->state;

src/newgrf_engine.h

➞

Show inline comments

@@ @@ -84,9 +84,9 @@ void GetCustomEngineSprite(EngineID engi @@
 #define GetCustomVehicleSprite(v, direction, image_type, result) GetCustomEngineSprite(v->engine_type, v, direction, image_type, result)
 #define GetCustomVehicleIcon(et, direction, image_type, result) GetCustomEngineSprite(et, nullptr, direction, image_type, result)
 void GetRotorOverrideSprite(EngineID engine, const struct Aircraft *v, bool info_view, EngineImageType image_type, VehicleSpriteSeq *result);
 #define GetCustomRotorSprite(v, i, image_type, result) GetRotorOverrideSprite(v->engine_type, v, i, image_type, result)
 #define GetCustomRotorIcon(et, image_type, result) GetRotorOverrideSprite(et, nullptr, true, image_type, result)
 void GetRotorOverrideSprite(EngineID engine, const struct Aircraft *v, EngineImageType image_type, VehicleSpriteSeq *result);
 #define GetCustomRotorSprite(v, image_type, result) GetRotorOverrideSprite(v->engine_type, v, image_type, result)
 #define GetCustomRotorIcon(et, image_type, result) GetRotorOverrideSprite(et, nullptr, image_type, result)
 /* Forward declaration of GRFFile, to avoid unnecessary inclusion of newgrf.h
  * elsewhere... */

src/vehicle_gui.cpp

➞

Show inline comments

@@ @@ -3191,7 +3191,7 @@ void SetMouseCursorVehicle(const Vehicle @@
 		VehicleSpriteSeq seq;
 		if (rotor_seq) {
-			GetCustomRotorSprite(Aircraft::From(v), true, image_type, &seq);
 			GetCustomRotorSprite(Aircraft::From(v), image_type, &seq);
 			if (!seq.IsValid()) seq.Set(SPR_ROTOR_STOPPED);
 			y_offset = - ScaleGUITrad(5);
 		} else {

0 comments (0 inline, 0 general)