Eliminate instance of code duplication

Author: MMulthaupt

cmd/launcher/gui/reporting.go

@@ -19,12 +19,6 @@ import (
 	"github.com/setlog/trivrost/cmd/launcher/flags"
 )
 
-func ReportFatalError(fatalError error, launcherFlags *flags.LauncherFlags) {
-	WaitUntilReady()
-	defer Quit()
-	PanicInformatively(fatalError, launcherFlags)
-}
-
 func HandlePanic(launcherFlags *flags.LauncherFlags) {
 	if r := recover(); r != nil {
 		if err, ok := r.(error); ok && errors.Is(err, context.Canceled) {

cmd/launcher/launcher/entry.go

@@ -9,14 +9,9 @@ import (
 	log "github.com/sirupsen/logrus"
 
 	"github.com/setlog/trivrost/cmd/launcher/flags"
-	"github.com/setlog/trivrost/cmd/launcher/gui"
 )
 
 func LauncherMain(ctx context.Context, launcherFlags *flags.LauncherFlags) {
-	gui.WaitUntilReady()
-	defer gui.Quit()
-	defer gui.HandlePanic(launcherFlags)
-
 	places.MakePlaces()
 	defer Linger()
 	locking.AcquireLock(ctx, launcherFlags)

cmd/launcher/main.go

@@ -45,25 +45,12 @@ var gitBranch string
 
 func main() {
 	defer misc.LogPanic()
-	launcherFlags, fatalError := initializeEnvironment()
+	launcherFlags, envErr := initializeEnvironment()
 	ctx, cancelFunc := context.WithCancel(context.Background())
 
-	if fatalError == nil {
-		go launcher.LauncherMain(ctx, launcherFlags)
-	} else {
-		go gui.ReportFatalError(fatalError, launcherFlags)
-	}
-
-	// On MacOS, only the first thread created by the OS is allowed to be the main GUI thread.
-	// Also, on Windows, OLE code needs to run on the main thread, which we rely on when creating shortcuts.
-	runtime.LockOSThread()
+	go runLauncher(ctx, envErr, launcherFlags)
+	runGUI(ctx, cancelFunc, launcherFlags, envErr == nil)
 
-	err := gui.Main(ctx, cancelFunc, resources.LauncherConfig.BrandingName, !launcherFlags.Uninstall && fatalError == nil)
-	if err != nil {
-		log.Fatalf("gui.Main() failed: %v\n", err)
-	}
-
-	log.Info("End of main().")
 	log.Exit(0)
 }
 
@@ -80,6 +67,28 @@ func initializeEnvironment() (*flags.LauncherFlags, error) {
 	return launcherFlags, misc.NewNestedErrorFromFirstCause(argumentError, flagError, pathError, placesError)
 }
 
+func runLauncher(ctx context.Context, fatalError error, launcherFlags *flags.LauncherFlags) {
+	gui.WaitUntilReady()
+	defer gui.Quit()
+	if fatalError != nil {
+		gui.PanicInformatively(fatalError, launcherFlags)
+	}
+	defer gui.HandlePanic(launcherFlags)
+	launcher.LauncherMain(ctx, launcherFlags)
+}
+
+func runGUI(ctx context.Context, cancelFunc context.CancelFunc, launcherFlags *flags.LauncherFlags, showMainWindow bool) {
+	// On MacOS, only the first thread created by the OS is allowed to be the main GUI thread.
+	// Also, on Windows, OLE code needs to run on the main thread, which we rely on when creating shortcuts.
+	runtime.LockOSThread()
+
+	err := gui.Main(ctx, cancelFunc, resources.LauncherConfig.BrandingName, !launcherFlags.Uninstall && showMainWindow)
+	if err != nil {
+		log.Fatalf("gui.Main() failed: %v\n", err)
+	}
+	log.Info("runGUI() terminated.")
+}
+
 func parseEnvironment() (launcherFlags *flags.LauncherFlags, argumentError, flagError, pathError, evalError, placesError error) {
 	launcherFlags = &flags.LauncherFlags{}
 	if len(os.Args) < 1 {