From 1564e00215e18dc3557ee86bae4a9d91e098c449 Mon Sep 17 00:00:00 2001 From: Copilot <198982749+Copilot@users.noreply.github.com> Date: Wed, 27 May 2026 09:35:54 -0700 Subject: [PATCH] smt2parser: realign `pop_app_frame` non-`expr_head` `else` block indentation (#9646) This updates a formatting regression introduced in the `pop_app_frame` non-`expr_head` path, where block indentation made control flow harder to read. The patch is whitespace-only and keeps parser behavior unchanged. - **What changed** - Reindented the `else` body in `src/parsers/smt2/smt2parser.cpp::pop_app_frame` so nested `if/else` structure is visually unambiguous. - Removed trailing spaces on the `m_ctx.mk_app(symbol("select"), ...)` lines in the same block. - **Scope** - No control-flow, data-flow, or API changes. - No changes outside `pop_app_frame`. ```cpp // Before else { local l; if (m_env.find(fr->m_f, l)) { ... } else { ... } } // After else { local l; if (m_env.find(fr->m_f, l)) { ... } else { ... } } ``` --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> --- src/parsers/smt2/smt2parser.cpp | 46 ++++++++++++++++----------------- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/src/parsers/smt2/smt2parser.cpp b/src/parsers/smt2/smt2parser.cpp index ca888b12d..a4e3a44a3 100644 --- a/src/parsers/smt2/smt2parser.cpp +++ b/src/parsers/smt2/smt2parser.cpp @@ -2008,32 +2008,32 @@ namespace smt2 { } } else { - local l; - if (m_env.find(fr->m_f, l)) { - push_local(l); - t_ref = expr_stack().back(); - for (unsigned i = 0; i < num_args; ++i) { - expr* arg = expr_stack().get(fr->m_expr_spos + i); - expr* args[2] = { t_ref.get(), arg }; - m_ctx.mk_app(symbol("select"), - 2, - args, - 0, - nullptr, - nullptr, + local l; + if (m_env.find(fr->m_f, l)) { + push_local(l); + t_ref = expr_stack().back(); + for (unsigned i = 0; i < num_args; ++i) { + expr* arg = expr_stack().get(fr->m_expr_spos + i); + expr* args[2] = { t_ref.get(), arg }; + m_ctx.mk_app(symbol("select"), + 2, + args, + 0, + nullptr, + nullptr, + t_ref); + } + } + else { + m_ctx.mk_app(fr->m_f, + num_args, + expr_stack().data() + fr->m_expr_spos, + num_indices, + m_param_stack.data() + fr->m_param_spos, + fr->m_as_sort ? sort_stack().back() : nullptr, t_ref); } } - else { - m_ctx.mk_app(fr->m_f, - num_args, - expr_stack().data() + fr->m_expr_spos, - num_indices, - m_param_stack.data() + fr->m_param_spos, - fr->m_as_sort ? sort_stack().back() : nullptr, - t_ref); - } - } expr_stack().shrink(fr->m_expr_spos); m_param_stack.shrink(fr->m_param_spos); if (fr->m_as_sort)