From 2ac0f56105d6e76b183db9b82d5aab5ac72134c8 Mon Sep 17 00:00:00 2001
From: altair823 <dlsrks0734@gmail.com>
Date: Thu, 30 Apr 2026 14:18:54 +0000
Subject: [PATCH] p1-3: drop dead bindings in heading + table end handlers
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Removes the leftover `let _ = level_u8;` and `let _ = header_count;`
discards. The Heading frame already carries the canonical level (we
populated it from `Tag::Heading` at Start), so we destructure that
directly and ignore the redundant `TagEnd::Heading(level)` payload.
The header_count helper was dead — Frame::Table tracks `cols`
internally and we never consumed `header_count`.
---
 crates/kb-parse-md/src/blocks.rs | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/crates/kb-parse-md/src/blocks.rs b/crates/kb-parse-md/src/blocks.rs
index 86913de..5d225c0 100644
--- a/crates/kb-parse-md/src/blocks.rs
+++ b/crates/kb-parse-md/src/blocks.rs
@@ -519,13 +519,13 @@ impl<'a> WalkState<'a> {
             }
 
             // ---- Container ends -------------------------------------------------
-            Event::End(TagEnd::Heading(level)) => {
-                let level_u8 = heading_level_to_u8(level);
-                if let Some(Frame::Heading { level: lvl, range, inlines }) = self.frames.pop() {
+            Event::End(TagEnd::Heading(_level)) => {
+                // The Tag::Heading frame is the source of truth for the
+                // level — `_level` from TagEnd is identical for well-formed
+                // input. We trust the frame.
+                if let Some(Frame::Heading { level: level_to_use, range, inlines }) = self.frames.pop() {
                     let (_inline_vec, text) = inlines.finish();
                     let text = text.trim().to_string();
-                    let level_to_use = lvl;
-                    let _ = level_u8;
 
                     // Update heading stack: clear deeper levels, set this level.
                     if (1..=6).contains(&level_to_use) {
@@ -679,12 +679,10 @@ impl<'a> WalkState<'a> {
                     range,
                     headers,
                     rows,
-                    cols,
                     malformed,
                     ..
                 }) = self.frames.pop()
                 {
-                    let header_count = if cols > 0 { cols } else { headers.len() };
                     let block = if let Some(note) = malformed {
                         // Fall back to a paragraph carrying the raw markdown.
                         warnings.push(Warning {
@@ -706,7 +704,6 @@ impl<'a> WalkState<'a> {
                             },
                         }
                     } else {
-                        let _ = header_count;
                         ParsedBlock {
                             kind: ParsedBlockKind::Table,
                             heading_path: self.heading_path(),