perf(core): drop the duplicate stale-lane pass in dropStaleSpans

commit: 02f068b689c9dcfaf8c1732f5f1cd3307b69a229
author: brice · 35h ago · 2026-10-21 08:21 UTC
parent: 09733c2
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +1 -13

M internal/core/layout_policy.go +1 -13


@@ -96,19 +96,7 @@ func (w *Workspace) dropStaleSpans() {
96	96		if !stamped {
97	97		return
98	98		}
99		-	// The common case has only valid lanes: find out without allocating.
100		-	anyStale := false
101		-	for first := 0; first < len(w.Columns); {
102		-	_, last := w.laneBounds(w.laneOf(first))
103		-	if !w.validSpans(first, last) {
104		-	anyStale = true
105		-	break
106		-	}
107		-	first = last + 1
108		-	}
109		-	if !anyStale {
110		-	return
111		-	}
	99	+	// A nil stale slice does not allocate, so valid lanes cost nothing here.
112	100		var stale []int
113	101		for first := 0; first < len(w.Columns); {
114	102		_, last := w.laneBounds(w.laneOf(first))


@@ -96,19 +96,7 @@ func (w *Workspace) dropStaleSpans() {
96		if !stamped {	96		if !stamped {
97		return	97		return
98		}	98		}
99	-	// The common case has only valid lanes: find out without allocating.	99	+	// A nil stale slice does not allocate, so valid lanes cost nothing here.
100	-	anyStale := false
101	-	for first := 0; first < len(w.Columns); {
102	-	_, last := w.laneBounds(w.laneOf(first))
103	-	if !w.validSpans(first, last) {
104	-	anyStale = true
105	-	break
106	-	}
107	-	first = last + 1
108	-	}
109	-	if !anyStale {
110	-	return
111	-	}
112		var stale []int	100		var stale []int
113		for first := 0; first < len(w.Columns); {	101		for first := 0; first < len(w.Columns); {
114		_, last := w.laneBounds(w.laneOf(first))	102		_, last := w.laneBounds(w.laneOf(first))