Refactor stats tracking and memory management in view modules

- Replace direct stats field increments with dedicated increment_*
methods in Stats
- Update all view constructors and generators (ContainerView, ImageView,
TextView, RtfView) to use new stats increment methods
- Add clear() methods to View and Layout for improved GC and memory
cleanup
- Refactor Window update logic: ensure proper clearing of views/layouts,
refresh UI, and use update_max_renderers for stats
- Minor improvements to stats output formatting

Author: mike-ward

stats.v

@@ -15,6 +15,32 @@ mut:
 	max_renderers   usize
 }
 
+fn (mut stats Stats) increment_container_views() {
+	stats.container_views += 1
+}
+
+fn (mut stats Stats) increment_text_views() {
+	stats.text_views += 1
+}
+
+fn (mut stats Stats) increment_image_views() {
+	stats.image_views += 1
+}
+
+fn (mut stats Stats) increment_rtf_views() {
+	stats.rtf_views += 1
+}
+
+fn (mut stats Stats) update_max_renderers(count usize) {
+	if count > stats.max_renderers {
+		stats.max_renderers = count
+	}
+}
+
+fn (mut stats Stats) increment_layouts() {
+	stats.layouts += 1
+}
+
 fn (window &Window) stats() string {
 	mut tx := []string{}
 	tx << ''
@@ -31,7 +57,7 @@ fn (window &Window) stats() string {
 fn (window &Window) view_stats() string {
 	mut tx := []string{}
 	tx << ''
-	tx << 'Views'
+	tx << 'Views Generated'
 	tx << stat_sub_div
 	tx << 'container views ${cm(gui_stats.container_views):17}'
 	tx << 'text views      ${cm(gui_stats.text_views):17}'

view.v

@@ -25,3 +25,21 @@ fn generate_layout(view &View, mut window Window) Layout {
 	}
 	return layout
 }
+
+// clear is feeble attempt to get the GC to collect views
+fn (mut view View) clear() {
+	for mut content in view.content {
+		content.clear()
+	}
+	unsafe { view.content.reset() }
+	view.content.clear()
+}
+
+// clear is feeble attempt to get the GC to collect layouts
+fn (mut layout Layout) clear() {
+	for mut child in layout.children {
+		child.clear()
+	}
+	unsafe { layout.children.reset() }
+	layout.children.clear()
+}

view_container.v

@@ -60,7 +60,7 @@ mut:
 fn (cv ContainerView) generate(mut _ Window) Layout {
 	assert cv.shape_type in [.rectangle, .circle]
 	$if !prod {
-		gui_stats.layouts += 1
+		gui_stats.increment_layouts()
 	}
 	if cv.invisible {
 		return Layout{}
@@ -224,6 +224,10 @@ pub:
 // its content top-to-bottom or left_to_right. A `.none` axis allows a
 // container to behave as a canvas with no additional layout.
 fn container(cfg ContainerCfg) View {
+	$if !prod {
+		gui_stats.increment_container_views()
+	}
+
 	if cfg.invisible {
 		return ContainerView{
 			over_draw: true // removes it from spacing calculations
@@ -278,10 +282,6 @@ fn container(cfg ContainerCfg) View {
 		else { cfg.content }
 	}
 
-	$if !prod {
-		gui_stats.container_views += 1
-	}
-
 	return ContainerView{
 		id:              cfg.id
 		id_focus:        cfg.id_focus

view_image.v

@@ -35,7 +35,7 @@ pub:
 
 fn (iv &ImageView) generate(mut window Window) Layout {
 	$if !prod {
-		gui_stats.layouts += 1
+		gui_stats.increment_layouts()
 	}
 	if iv.invisible {
 		return Layout{}
@@ -68,7 +68,7 @@ fn (iv &ImageView) generate(mut window Window) Layout {
 
 pub fn image(cfg ImageCfg) View {
 	$if !prod {
-		gui_stats.image_views += 1
+		gui_stats.increment_image_views()
 	}
 	return ImageView{
 		id:         cfg.id

view_rtf.v

@@ -40,7 +40,7 @@ pub:
 
 fn (rtf &RtfView) generate(mut window Window) Layout {
 	$if !prod {
-		gui_stats.layouts += 1
+		gui_stats.increment_layouts()
 	}
 
 	if rtf.invisible {
@@ -79,13 +79,13 @@ fn (rtf &RtfView) generate(mut window Window) Layout {
 
 // rtf creates a view from the given [RtfCfg](#RtfCfg)
 pub fn rtf(cfg RtfCfg) View {
-	mut ll := datatypes.LinkedList[TextSpan]{}
-	ll.push_many(cfg.spans)
-
 	$if !prod {
-		gui_stats.rtf_views += 1
+		gui_stats.increment_rtf_views()
 	}
 
+	mut ll := datatypes.LinkedList[TextSpan]{}
+	ll.push_many(cfg.spans)
+
 	return RtfView{
 		id:         cfg.id
 		id_focus:   cfg.id_focus

view_text.v

@@ -21,7 +21,7 @@ mut:
 
 fn (t &TextView) generate(mut window Window) Layout {
 	$if !prod {
-		gui_stats.layouts += 1
+		gui_stats.increment_layouts()
 	}
 	if t.cfg.invisible {
 		return Layout{}
@@ -105,7 +105,7 @@ fn (t &TextCfg) free() {
 // operations. See [TextCfg](#TextCfg)
 pub fn text(cfg TextCfg) View {
 	$if !prod {
-		gui_stats.text_views += 1
+		gui_stats.increment_text_views()
 	}
 	return TextView{
 		cfg:    &cfg

window.v

@@ -108,6 +108,7 @@ pub fn window(cfg &WindowCfg) &Window {
 				w.blinky_cursor_animation()
 			}
 			on_init(w)
+			w.update_window()
 		}
 		sample_count:                 int(cfg.samples)
 	)
@@ -128,12 +129,12 @@ fn frame_fn(mut window Window) {
 	window.ui.begin()
 	renderers_draw(window.renderers, window)
 	window.ui.end()
-	gc_collect() // revisit gc_collect() once leak is found. Strikes me as a performance issue maybe - mrw
 	sapp.set_mouse_cursor(window.view_state.mouse_cursor)
 	$if trace_update_window_calls ? {
 		println(window.update_window_calls)
 	}
 	window.update_window_calls = 0
+	gc_collect() // revisit gc_collect() once leak is found. Strikes me as a performance issue maybe - mrw
 	window.unlock()
 }
 
@@ -227,53 +228,37 @@ fn event_fn(ev &gg.Event, mut w Window) {
 // and replaces the current view generator.
 pub fn (mut window Window) update_view(gen_view fn (&Window) View) {
 	// Order matters here. Clear the view state first
-	window.view_state.clear(mut window)
-	view := gen_view(window)
-	mut layout := window.compose_layout(view)
-
 	window.lock()
+	window.view_state.clear(mut window)
 	window.view_generator = gen_view
-	window.layout = layout
-	window.renderers.clear()
-	clip_rect := window.window_rect()
-	background := window.color_background()
-	render_layout(mut layout, background, clip_rect, mut window)
-
-	$if !prod {
-		if window.renderers.len > gui_stats.max_renderers {
-			gui_stats.max_renderers = usize(window.renderers.len)
-		}
-	}
-
 	window.unlock()
-	window.ui.refresh_ui()
 }
 
 // update_window generates a new layout from the window's current
 // view generator.
-pub fn (mut window Window) update_window() {
+fn (mut window Window) update_window() {
 	window.update_window_calls++
 	if window.update_window_calls > window.max_update_window_calls_per_frame {
 		return
 	}
 
 	window.lock()
-
-	view := window.view_generator(window)
+	mut view := window.view_generator(window)
+	mut old_layout := window.layout
 	window.layout = window.compose_layout(view)
 	window.renderers.clear()
 	clip_rect := window.window_rect()
 	background := window.color_background()
 	render_layout(mut window.layout, background, clip_rect, mut window)
+	window.unlock()
+
+	view.clear()
+	old_layout.clear()
+	window.ui.refresh_ui()
 
 	$if !prod {
-		if window.renderers.len > gui_stats.max_renderers {
-			gui_stats.max_renderers = usize(window.renderers.len)
-		}
+		gui_stats.update_max_renderers(usize(window.renderers.len))
 	}
-
-	window.unlock()
-	window.ui.refresh_ui()
 }
 
 // compose_layout produces a layout from the given view that is