Fixed loading error/warning screen not showing (#6218)

patches/minecraft/net/minecraft/client/Minecraft.java.patch

@@ -81,7 +81,7 @@
              this.func_213256_aB();
           }
 -
-+         net.minecraftforge.fml.client.ClientModLoader.complete();
++         if (net.minecraftforge.fml.client.ClientModLoader.completeModLoading()) return; // Do not overwrite the error sceen
 +         // FORGE: Move opening initial screen to after startup and events are enabled.
 +         // Also Fixes MC-145102
 +         if (this.field_71475_ae != null) {

src/main/java/net/minecraftforge/fml/client/ClientModLoader.java

@@ -142,7 +142,13 @@ public class ClientModLoader
         return VersionChecker.Status.UP_TO_DATE;
     }
 
+    @Deprecated // TODO: remove in 1.15
     public static void complete()
+    {
+        completeModLoading();
+    }
+
+    public static boolean completeModLoading()
     {
         GlStateManager.disableTexture();
         GlStateManager.enableTexture();
@@ -161,15 +167,17 @@ public class ClientModLoader
             }
             warnings = Collections.emptyList(); //Clear warnings, as the user does not want to see them
         }
-        if (error != null || !warnings.isEmpty()) {
-            mc.displayGuiScreen(new LoadingErrorScreen(error, warnings));
-        } else {
-            ClientHooks.logMissingTextureErrors();
-        }
         if (error == null) {
             // We can finally start the forge eventbus up
             MinecraftForge.EVENT_BUS.start();
         }
+        if (error != null || !warnings.isEmpty()) {
+            mc.displayGuiScreen(new LoadingErrorScreen(error, warnings));
+            return true;
+        } else {
+            ClientHooks.logMissingTextureErrors();
+            return false;
+        }
     }
 
     public static void renderProgressText() {