From 0884aad1c56d2938dad7449232cfe25bc833669f Mon Sep 17 00:00:00 2001 From: Piotr Wilkin Date: Sat, 14 Feb 2026 00:37:33 +0100 Subject: [PATCH] Remove stupid LLM-generated method comment headers (yeah, we can see what the method name is, thank you very much) --- common/chat-auto-parser-generator.cpp | 16 ---------------- common/chat-diff-analyzer.cpp | 16 ---------------- 2 files changed, 32 deletions(-) diff --git a/common/chat-auto-parser-generator.cpp b/common/chat-auto-parser-generator.cpp index bc2b1f7bbe..072bdb795f 100644 --- a/common/chat-auto-parser-generator.cpp +++ b/common/chat-auto-parser-generator.cpp @@ -74,10 +74,6 @@ common_chat_params universal_peg_generator::generate_parser(const common_chat_te return data; } -// ============================================================================ -// analyze_template::build_parser - orchestrates parser building -// ============================================================================ - common_peg_arena analyze_template::build_parser(const templates_params & inputs) const { return build_chat_peg_unified_parser([&](common_chat_peg_unified_builder & p) { p.set_allow_python_dict_format(true); @@ -107,10 +103,6 @@ common_peg_arena analyze_template::build_parser(const templates_params & inputs) }); } -// ============================================================================ -// analyze_reasoning::build_parser -// ============================================================================ - common_peg_parser analyze_reasoning::build_parser(parser_build_context & ctx) const { auto & p = ctx.p; @@ -139,10 +131,6 @@ common_peg_parser analyze_reasoning::build_parser(parser_build_context & ctx) co return p.eps(); } -// ============================================================================ -// analyze_content::build_parser -// ============================================================================ - common_peg_parser analyze_content::build_parser(parser_build_context & ctx) const { auto & p = ctx.p; @@ -164,10 +152,6 @@ common_peg_parser analyze_content::build_optional_wrapped(parser_build_context & return p.eps(); } -// ============================================================================ -// analyze_tools::build_parser - dispatches to format-specific builders -// ============================================================================ - common_peg_parser analyze_tools::build_parser(parser_build_context & ctx) const { switch (format.mode) { case tool_format::JSON_NATIVE: diff --git a/common/chat-diff-analyzer.cpp b/common/chat-diff-analyzer.cpp index 2256e48976..b49ce22ef4 100644 --- a/common/chat-diff-analyzer.cpp +++ b/common/chat-diff-analyzer.cpp @@ -138,10 +138,6 @@ static json second_tool_call = static json first_tool_call_alt_id = build_tool_call("foofoo", json{{ "first", "XXXX" }, { "second", "YYYY" }}, "call99999"); -// ============================================================================ -// analyze_template -// ============================================================================ - analyze_template::analyze_template(const common_chat_template & tmpl) : jinja_caps(tmpl.original_caps()) , reasoning(tmpl, jinja_caps.supports_tool_calls) @@ -192,10 +188,6 @@ void analyze_template::collect_preserved_tokens() { add_token(tools.call_id.suffix); } -// ============================================================================ -// analyze_reasoning -// ============================================================================ - analyze_reasoning::analyze_reasoning(const common_chat_template & tmpl, bool supports_tools) : analyze_base(tmpl) { LOG_DBG(ANSI_ORANGE "Phase 1: Reasoning analysis\n" ANSI_RESET); @@ -454,10 +446,6 @@ void analyze_reasoning::compare_reasoning_scope() { } } -// ============================================================================ -// analyze_content -// ============================================================================ - analyze_content::analyze_content(const common_chat_template & tmpl, const analyze_reasoning & reasoning) : analyze_base(tmpl) { LOG_DBG(ANSI_ORANGE "Phase 2: Content analysis\n" ANSI_RESET); @@ -549,10 +537,6 @@ bool analyze_content::is_always_wrapped() const { return mode == content_mode::ALWAYS_WRAPPED && !start.empty() && !end.empty(); } -// ============================================================================ -// analyze_tools -// ============================================================================ - analyze_tools::analyze_tools(const common_chat_template & tmpl, const jinja::caps & caps, const analyze_reasoning & reasoning)