From a9870f81489c24c50e9b6735a64b20a0925f2f31 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=CE=B6eh=20Matt?= <5415177+ZehMatt@users.noreply.github.com> Date: Fri, 1 Mar 2024 17:18:05 +0200 Subject: [PATCH 1/5] Fix #21484: Initialize drops to zero when creating a ride --- src/openrct2/actions/RideCreateAction.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/openrct2/actions/RideCreateAction.cpp b/src/openrct2/actions/RideCreateAction.cpp index c67a2e2959..136a67f28c 100644 --- a/src/openrct2/actions/RideCreateAction.cpp +++ b/src/openrct2/actions/RideCreateAction.cpp @@ -308,6 +308,7 @@ GameActions::Result RideCreateAction::Execute() const ride->income_per_hour = kMoney64Undefined; ride->profit = kMoney64Undefined; ride->connected_message_throttle = 0; + ride->drops = 0; ride->entrance_style = OBJECT_ENTRY_INDEX_NULL; if (rtd.HasFlag(RIDE_TYPE_FLAG_HAS_ENTRANCE_EXIT)) From 9a8ca8e2d15c9655e0e5034311bd9be8ad07469e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=CE=B6eh=20Matt?= <5415177+ZehMatt@users.noreply.github.com> Date: Fri, 1 Mar 2024 17:29:46 +0200 Subject: [PATCH 2/5] Update changelog.txt --- distribution/changelog.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/distribution/changelog.txt b/distribution/changelog.txt index 663f20fa3b..31f689585b 100644 --- a/distribution/changelog.txt +++ b/distribution/changelog.txt @@ -32,6 +32,7 @@ - Fix: [#21347] Too many options are hidden if the platform has no file picker. - Fix: [#21350] Maze and Mini Golf track designs from RCT1 not shown in track designs list. - Fix: [#21425] Additional missing/misplaced land & construction rights tiles in Japanese Coastal Reclaim. +- Fix: [#21484] Upkeep costs for some rides/facilities/shops are not calculated correctly. - Fix: [#21498] Crash when the size of text can’t be determined. - Fix: [objects#262, objects#263, objects#265, objects#266, objects#267, objects#268, objects#270, objects#271, objects#283] Various errors in expansion pack objects (original bug). - Fix: [OpenSFX#18] B&M Roar sound effect not looping correctly. From 78406f4d671e73c2ca57d6eff867ba913268a80c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=CE=B6eh=20Matt?= <5415177+ZehMatt@users.noreply.github.com> Date: Fri, 1 Mar 2024 17:30:25 +0200 Subject: [PATCH 3/5] Bump up network version --- src/openrct2/network/NetworkBase.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/openrct2/network/NetworkBase.cpp b/src/openrct2/network/NetworkBase.cpp index 13b6833a7e..c3d8c5d504 100644 --- a/src/openrct2/network/NetworkBase.cpp +++ b/src/openrct2/network/NetworkBase.cpp @@ -46,7 +46,7 @@ using namespace OpenRCT2; // It is used for making sure only compatible builds get connected, even within // single OpenRCT2 version. -#define NETWORK_STREAM_VERSION "4" +#define NETWORK_STREAM_VERSION "5" #define NETWORK_STREAM_ID OPENRCT2_VERSION "-" NETWORK_STREAM_VERSION From 9bfc3a0a29a622503a4ba6add5b067b90111de88 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=CE=B6eh=20Matt?= <5415177+ZehMatt@users.noreply.github.com> Date: Fri, 1 Mar 2024 18:17:16 +0200 Subject: [PATCH 4/5] Zero initialize some fields in ride station --- src/openrct2/actions/RideCreateAction.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/openrct2/actions/RideCreateAction.cpp b/src/openrct2/actions/RideCreateAction.cpp index 136a67f28c..42894b6a4d 100644 --- a/src/openrct2/actions/RideCreateAction.cpp +++ b/src/openrct2/actions/RideCreateAction.cpp @@ -148,6 +148,10 @@ GameActions::Result RideCreateAction::Execute() const station.Exit.SetNull(); station.TrainAtStation = RideStation::NO_TRAIN; station.QueueTime = 0; + station.SegmentLength = 0; + station.QueueLength = 0; + station.Length = 0; + station.Height = 0; } std::fill(std::begin(ride->vehicles), std::end(ride->vehicles), EntityId::GetNull()); From 556004bf37253383e7e22f506dcdb76c87c44e81 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=CE=B6eh=20Matt?= <5415177+ZehMatt@users.noreply.github.com> Date: Fri, 1 Mar 2024 18:32:04 +0200 Subject: [PATCH 5/5] Filter NETWORK_STREAM_VERSION from clang-tidy --- .clang-tidy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.clang-tidy b/.clang-tidy index f9c1c33605..7434d0073b 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -6,6 +6,6 @@ Checks: > modernize-use-override CheckOptions: - key: cppcoreguidelines-macro-usage.AllowedRegexp - value: 'validate_global_widx' + value: 'validate_global_widx|NETWORK_STREAM_VERSION' WarningsAsErrors: true FormatStyle: 'file'