altair823-org/kebab
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 45 Wiki Activity

p1-3: drop dead bindings in heading + table end handlers

Browse Source 
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`.
This commit is contained in:
altair823
2026-04-30 14:18:54 +00:00
parent f604a381df
commit 2ac0f56105
1 changed files with 5 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
crates/kb-parse-md/src/blocks.rs
View File

@@ -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(),