performance: slight performance improvement when handling modules => …

…only sort onces every module is done

internal/modules/finder.go

@@ -66,18 +66,18 @@ func (f *Finder) SetupData(cfg *config.Config, ctx context.Context) {
 
 	fileWalker.SetErrorHandler(errorHandler)
 
-	isWalking := make(chan bool)
+	done := make(chan bool)
 
 	go func(isWalking chan bool) {
 		err := fileWalker.Start()
 		if err == nil {
 			isWalking <- false
 		}
-	}(isWalking)
+	}(done)
 
 	for {
 		select {
-		case <-isWalking:
+		case <-done:
 			fileWalker.Terminate()
 			f.general.IsSetup = true
 			f.general.HasInitialSetup = true

internal/ui/handler.go

@@ -7,7 +7,6 @@ import (
 	"sync"
 
 	"github.com/abenz1267/walker/internal/util"
-	"github.com/diamondburned/gotk4/pkg/glib/v2"
 )
 
 type Handler struct {
@@ -28,21 +27,6 @@ func (h *Handler) handle() {
 
 			h.mut.Lock()
 			h.entries = append(h.entries, entries...)
-
-			if !appstate.KeepSort && !h.keepSort {
-				sortEntries(h.entries)
-			}
-
-			if len(h.entries) > cfg.List.MaxEntries {
-				h.entries = h.entries[:cfg.List.MaxEntries]
-			}
-
-			if len(h.entries) > 0 {
-				glib.IdleAdd(func() {
-					common.items.Splice(0, int(common.items.NItems()), h.entries...)
-				})
-			}
-
 			h.mut.Unlock()
 		case <-h.ctx.Done():
 			return

internal/ui/interactions.go

@@ -736,6 +736,24 @@ func processAsync(ctx context.Context, text string) {
 
 	wg.Wait()
 
+	handler.mut.Lock()
+
+	if !appstate.KeepSort && !handler.keepSort {
+		sortEntries(handler.entries)
+	}
+
+	if len(handler.entries) > cfg.List.MaxEntries {
+		handler.entries = handler.entries[:cfg.List.MaxEntries]
+	}
+
+	if len(handler.entries) > 0 {
+		glib.IdleAdd(func() {
+			common.items.Splice(0, int(common.items.NItems()), handler.entries...)
+		})
+	}
+
+	handler.mut.Unlock()
+
 	if !layout.Window.Box.Search.Spinner.Hide {
 		elements.spinner.SetVisible(false)
 	}