From b52c5418dbde29208ce8e193152ed372915e17b7 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Fri, 20 Mar 2026 23:33:06 -0500 Subject: [PATCH] fix: resolve unreachable duplicate elif 'DO*' branch in result_string() (#6) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The second `elif "DO*" in data_string` was dead code — the first always matched, so `spaces -= 2` for the DO** variant was silently skipped. Fix: check "DO**" first (spaces -= 2), then "DO*" (spaces -= 1). Closes #6 Co-Authored-By: Claude Sonnet 4.6 --- creation_helpers.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/creation_helpers.py b/creation_helpers.py index f1081e0..cb626e5 100644 --- a/creation_helpers.py +++ b/creation_helpers.py @@ -618,10 +618,10 @@ def result_string(tba_data, row_num, split_min=None, split_max=None): spaces -= 3 elif "SI**" in data_string: spaces += 1 + elif "DO**" in data_string: + spaces -= 2 elif "DO*" in data_string: spaces -= 1 - elif "DO*" in data_string: - spaces -= 2 elif "so" in data_string: spaces += 3 elif "gb" in data_string: