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.

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

Master to testing #1088

Merged
merged 46 commits into from
Apr 13, 2021
Merged

Master to testing #1088

merged 46 commits into from
Apr 13, 2021

Conversation

krzychu124
Copy link
Member

@krzychu124 krzychu124 commented Apr 13, 2021

To minimize number of changes and simplify review master -> testing
After merge testing goes back to master - hopefully without conflicts and no need for review.

Conflicts:

  • TLM/TLM/Custom/AI/CustomTrainAI.cs (not exist anymore)
  • TLM/TLM/Custom/PathFinding/CustomPathFind.cs
  • TLM/TLM/Custom/PathFinding/CustomPathManager.cs
  • TLM/TLM/Patcher.cs (moved to lifecycle)
  • TLM/TLM/TLM.csproj
  • TLM/TLM/ThreadingExtension.cs (moved to lifecycle)
  • TLM/TLM/TrafficManagerMod.cs (moved to lifecycle)

TianQiBuTian and others added 30 commits August 30, 2020 15:34
…instead.

- use of lock statement instead Monitor directly.
- removed commented out debug.log calls.
- formatting

(cherry picked from commit c878dd2)
(cherry picked from commit 4d281c8)
…s version causes sometimes at rebuild strange white pixel artifacts.

(cherry picked from commit 4544d30)
#869: Apply Shift+Click speed limit to entire roundabout

(cherry picked from commit ec3227d)
…l - set current tool to None, update OSD

(cherry picked from commit 3c0b6e4)

Road selection Panel - close on single right click, but leave menu open

(cherry picked from commit 39087f6)
…ts used to feed GUILayout.Window.

This will reduce allocations when some UI tools are open.
* made lifecycle functions/variables static and removed OnCreated OnRelease for better hot reload.
* cleanup lifecycle/Patcher.
Fixed SA1410 warnings by replacing the delegates with lambdas.
Fixed SA1137 Elements should have the same indentation warnings.
# Conflicts:
#	TLM/TLM/UI/SubTools/VehicleRestrictionsTool.cs
…ayoutWindow

Fixed all instances of delegate instance allocation (HAA0603) where i…
This is just a 11.5.1-hotfix1 merge with strategy=ours into master.
…d-in-tool-disabled

Hide overlays if no longer necessary
krzychu124 and others added 16 commits April 11, 2021 19:29
Main Camera cache reinitialization to prevent use of inactive camera
* encapsulate code for the lifecycle of CustomPathManager.
* restore old path manager and its path finds when unloading.
* hot unload now works
Harmony patcher continues on error (#1074).
Also, it logs which functions have been patched failed patching (#1075).
Moved all lifecycle code to a TMPELifeCycle.
# Conflicts:
#	TLM/TLM/UI/Textures/RoadUI.cs
#	TLM/TLM/UI/Textures/TrafficLightTextures.cs
SA1600 Elements should be documented
SA1602 Enumeration items should be documented
SA1611 Element parameters should be documented
SA1615 Element return value should be documented
SA1623 The property's documentation summary text should begin with: 'Gets'
…es-Traffic-Manager-President-Edition into master-to-testing

� Conflicts:
�	TLM/TLM/Custom/AI/CustomTrainAI.cs
�	TLM/TLM/Custom/PathFinding/CustomPathFind.cs
�	TLM/TLM/Custom/PathFinding/CustomPathManager.cs
�	TLM/TLM/Patcher.cs
�	TLM/TLM/TLM.csproj
�	TLM/TLM/ThreadingExtension.cs
�	TLM/TLM/TrafficManagerMod.cs
Copy link
Collaborator

@kvakvs kvakvs left a comment

Choose a reason for hiding this comment

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

Too long didn't read!

Copy link
Contributor

@DaEgi01 DaEgi01 left a comment

Choose a reason for hiding this comment

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

<3
so much harmony around here

@krzychu124 krzychu124 merged commit 54d3a8f into testing Apr 13, 2021
@krzychu124 krzychu124 deleted the master-to-testing branch April 13, 2021 23:59
@originalfoo originalfoo added this to the 11.6.0 milestone Jan 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants