Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ride entrance/exit may be built in the same position over 85.5m(=57 units) #5249

Closed
2 tasks
telk5093 opened this issue Feb 22, 2017 · 1 comment
Closed
2 tasks

Ride entrance/exit may be built in the same position over 85.5m(=57 units) #5249

telk5093 opened this issue Feb 22, 2017 · 1 comment
Assignees
@Gymnasiast
Labels
bug Something went wrong.

Comments

@telk5093
Copy link
Contributor

OS: Windows 10
Version: 0.0.7
Commit/Build: 7005d5d

It is issued with #5248 together in korean community,
ride entrance/exit could be built in the same position if their height is over 85.5m (=57 unit).

  • Reproducible in RCT2 (vanilla)?
  • Multiplayer?

Steps to reproduce:

  1. Disable support limit
  2. Build any ride with height as 85.5m (=57 unit)
  3. Then the clearance check of their entrance/exit isn't checked correctly. They could be built in the same position

Dump file

Screenshots / Video:
image

Save game:
@Gymnasiast
Copy link
Member

Gymnasiast commented Feb 23, 2017 •
edited

+57 units is internally represented as 128*. Now, if that number is represented as a signed 8-bit integer, the maximum value is 128. It will overflow if the game tries to increase it.

* You get the number by adding 7, then multiplying it by 2.

@Gymnasiast Gymnasiast mentioned this issue Feb 23, 2017
Closed
2 tasks
@Gymnasiast Gymnasiast self-assigned this Apr 11, 2017
@Gymnasiast Gymnasiast added the bug Something went wrong. label Apr 11, 2017
Gymnasiast added a commit to Gymnasiast/OpenRCT2 that referenced this issue Dec 29, 2019
@Gymnasiast
Fix OpenRCT2#5249: No collision detection when building ride entrance…
2c4a0f2 
… at heights > 85.5m
Gymnasiast added a commit to Gymnasiast/OpenRCT2 that referenced this issue Dec 29, 2019
@Gymnasiast
Fix OpenRCT2#5249: No collision detection when building ride entrance…
f9820db 
… at heights > 85.5m
Gymnasiast added a commit to Gymnasiast/OpenRCT2 that referenced this issue Dec 30, 2019
@Gymnasiast
Fix OpenRCT2#5249: No collision detection when building ride entrance…
ad00f14 
… at heights > 85.5m
janisozaur added a commit that referenced this issue Mar 24, 2020
@janisozaur
Release 0.2.5
4f6e77e 
- Feature: [#3154] Use a random title sequence each time it is shown.
- Feature: [#6553] Android version now runs in full screen.
- Feature: [#7865] Transport rides can now be synchronised.
- Feature: [#9073] Shortcut keys for the Tile Inspector.
- Feature: [#10305] Add two shortcuts for increasing and decreasing the scaling factor.
- Feature: [#10189] Make Track Designs work in multiplayer.
- Feature: [#10357] Added window for scenery scatter tool, allowing for area and density selection.
- Feature: [#10637] Console command to remove all floating objects.
- Change: [#1164] Use available translations for shortcut key bindings.
- Change: [#10997] Speed is automatically reset to normal upon scenario completion.
- Fix: [#2485] Hide Vertical Faces not applied to the edges of water.
- Fix: [#5249] No collision detection when building ride entrance at heights > 85.5m.
- Fix: [#6766] Changelog window doesn't open on some platforms.
- Fix: [#7784] Vehicle tab takes 1st car colour instead of tab_vehicle's colour.
- Fix: [#7854] Cannot build a custom spiral roller coaster design.
- Fix: [#7854] Empty entries in spiral roller coaster designs list.
- Fix: [#8151] Game freezes upon demolishing mazes at odd heights.
- Fix: [#8875] RCT1 competition scenarios are classified incorrectly.
- Fix: [#10176] Mistake in the sprite for the land tool's 6x6 grid.
- Fix: [#10196] Doors unable to be placed at end of track corners.
- Fix: [#10228] Can't import RCT1 Deluxe from Steam.
- Fix: [#10313] Path furniture can be placed on level crossings.
- Fix: [#10325] Crash when banners have no text.
- Fix: [#10376] No ratings generated when a shop and track intersect.
- Fix: [#10420] Money effect causing false positive desync.
- Fix: [#10477] Large Scenery cannot be placed higher using SHIFT.
- Fix: [#10489] Hosts last player action not being synchronized.
- Fix: [#10543] Secondary shop item prices are not imported correctly from RCT1 saves.
- Fix: [#10547] RCT1 parks have too many rides available.
- Fix: [#10587] Update last action coordinates on correct player.
- Fix: [#10631] Game bugs out and crashes if you get too many stations via copying stations with the tile inspector.
- Fix: [#10662] Duck cheat tooltips look odd and do not explain anything.
- Fix: [#10694] The lift hill speed of the flying roller coaster cannot be changed (original bug).
- Fix: [#10705] Apply multithreaded rendering to all viewports.
- Fix: [#10739] Mountain tool overlay for even-numbered selections.
- Fix: [#10752] Mute button state not correctly set at startup.
- Fix: [#10822] Can place too many peep spawns.
- Fix: [#10898] Banner text has an offset in tile inspector window.
- Fix: [#10904] RCT1/LL-scenarios with red water won't open.
- Fix: [#10941] The Clear Scenery tool gives refunds for ghost elements.
- Fix: [#10963] Light effects are drawn off-centre in some rotations.
- Fix: [#10993] Bottom toolbar not refreshing when a guest leaves the park.
- Fix: [#11001] Rides list does not use natural sorting.
- Fix: [objects#54] Stage Coach cars are not considered covered by the game.
- Fix: [objects#56] Handymen cut grass incorrectly.
- Improved: [#682] The staff patrol area is now drawn on the water, instead of on the surface under water.
- Improved: [#10858] Added horizontal grid lines to finance charts.
- Improved: [#10884] Added y-axes and labels to park window charts.
- Improved: [#10970] Introduced optional light effects for vehicles at night.
- Removed: [#6898] LOADMM and LOADRCT1 title sequence commands (use LOADSC instead).
XplosiveLugnut pushed a commit to XplosiveLugnut/OpenRCT2 that referenced this issue Apr 27, 2020
@Gymnasiast
Fix OpenRCT2#5249: No collision detection when building ride entrance…
9b18b35 
… at heights > 85.5m (OpenRCT2#10473)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Gymnasiast Gymnasiast

Labels
bug Something went wrong.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Gymnasiast @telk5093