Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.
And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.
patch spawn/despawn vehicle using new harmony
See original GitHub issuethis code is commented out: https://github.com/CitiesSkylinesMods/TMPE/blob/37b71da7e6f6793db1f2b579f14d96df02c7debf/TLM/TLM/Patch/_Vehicle/SpawnPatch.cs#L12 https://github.com/CitiesSkylinesMods/TMPE/blob/37b71da7e6f6793db1f2b579f14d96df02c7debf/TLM/TLM/Patch/_Vehicle/UnspawnPatch.cs#L12
Once I moved to harmony 2 should we uncomment this?
VehicleStateManager no longer exists but we can call OnSpawnVehicle(); instead. that method is already being called from CustomVehicle.CustomSpawn() so maybe I should delete Vehicle.Spawn/DespawnPatch
Issue Analytics
- State:
- Created 3 years ago
- Comments:21 (10 by maintainers)
Top Results From Across the Web
Top Related Medium Post
Top Related StackOverflow Question
Troubleshoot Live Code
Top Related Reddit Thread
Top Related Hackernoon Post
Top Related Tweet
Top Related Dev.to Post
Top Related Hashnode Post
Re:
StockPathFind.cs: We keep it so we can do diffs when new game version is released.BTW, I looking at the code in screenshots above, note that there are two different call contexts:
CustomSpawn()is called for vehicle and each trailer (if applicable) - ie. it can be called more than once per invocation of parent methodOnSpawnVehicle()is called only once, regardless of how many trailersSo I think we need two postfixes? One for the vanilla
.Spawn()and one for whatever the parent method is of the code in the screenshots above.TMPE is now harmony only.