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

DI Spreading: EngineSubsystems, StateMainMenu, StateHeadlessSetup, StateLoading #4201

Closed
wants to merge 12 commits into from

Conversation

DarkWeird
Copy link
Contributor

@DarkWeird DarkWeird commented Oct 15, 2020

Contains

Prepare for gestalt-di and removing CoreRegistry.
Contains Di spread for EngineSubsystems, StateMainMenu, StateHeadlessSetup, StateLoading

How to test

  1. Start game
  2. Create world
  3. run/dig a bit
  4. exit to main menu
  5. Host Game, load world
  6. run/dig a bit
  7. exit to main menu
  8. start dedicated server
    9 Join to Server
  9. run/dig a bit
  10. exit game

@lgtm-com
Copy link

lgtm-com bot commented Oct 15, 2020

This pull request introduces 1 alert when merging 0218fe8 into a829df4 - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@DarkWeird DarkWeird changed the title Feature/di spreading DI Spreading: EngineSubsystems, StateMainMenu, StateHeadlessSetup, StateLoading Oct 16, 2020
…AwareWorldProvider.

both implement WorldProviderCore but exposes only WorldProviderImpl. also WorldProviderWrapper must use EntityAwareProvider (context provides only WorldProviderCoreImpl as WorldPoviderCore)
# Conflicts:
#	engine/src/main/java/org/terasology/engine/subsystem/lwjgl/LwjglAudio.java
#	engine/src/main/java/org/terasology/engine/subsystem/lwjgl/LwjglGraphics.java
#	engine/src/main/java/org/terasology/engine/subsystem/lwjgl/LwjglInput.java
#	engine/src/main/java/org/terasology/rendering/backdrop/Skysphere.java
#	engine/src/main/java/org/terasology/rendering/nui/internal/LwjglCanvasRenderer.java
@pollend pollend closed this Feb 13, 2021
@skaldarnar skaldarnar deleted the feature/di-spreading branch June 5, 2021 22:38
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.

2 participants