Skip to content

Implement OpenLoco::Scenario::sub_4748D4 (0x004748D4)#3164

Merged
AaronVanGeffen merged 2 commits intoOpenLoco:masterfrom
petergaal:implement_sub_4748D4
Jul 25, 2025
Merged

Implement OpenLoco::Scenario::sub_4748D4 (0x004748D4)#3164
AaronVanGeffen merged 2 commits intoOpenLoco:masterfrom
petergaal:implement_sub_4748D4

Conversation

@petergaal
Copy link
Copy Markdown
Contributor

No description provided.

@AaronVanGeffen AaronVanGeffen merged commit d1fd493 into OpenLoco:master Jul 25, 2025
10 checks passed
@AaronVanGeffen AaronVanGeffen added this to the v25.07+ milestone Jul 25, 2025
@AaronVanGeffen
Copy link
Copy Markdown
Member

This appears to cause the landscape generation function to crash immediately. I'll investigate.

ObjectManager::sub_47966E();
ObjectManager::updateDefaultLevelCrossingType();
ObjectManager::sub_47AC05();
CompanyManager::sub_4A6DA9();
Copy link
Copy Markdown
Member

@AaronVanGeffen AaronVanGeffen Jul 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I replace this line with call(0x004A6DA9); the crash goes away. This suggests the issue is with #3084, and the current PR has only made the issue visible. Let's continue conversation there.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it still crashing after your changes? Or is it fixed now?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All fixed now

@petergaal petergaal deleted the implement_sub_4748D4 branch July 29, 2025 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants