From 0ca567cecd8b7a505f338dd64ab01889ff0bff06 Mon Sep 17 00:00:00 2001 From: Badvision Date: Sat, 25 Apr 2026 17:54:10 +0200 Subject: [PATCH] Fix break with no args to list breakpoints instead of printing usage Previously 'break' / 'bp' with no arguments printed a usage message. Now it lists active breakpoints, consistent with how monitor commands are expected to behave when invoked without arguments. Co-Authored-By: Claude Sonnet 4.6 --- src/main/java/jace/terminal/MainMode.java | 2 +- src/main/java/jace/terminal/MonitorMode.java | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/main/java/jace/terminal/MainMode.java b/src/main/java/jace/terminal/MainMode.java index 8ee6a2c..438b6d2 100644 --- a/src/main/java/jace/terminal/MainMode.java +++ b/src/main/java/jace/terminal/MainMode.java @@ -168,7 +168,7 @@ public class MainMode implements TerminalMode { commandHelp.put("cpu", "Displays the current CPU state (registers and flags).\nUsage: cpu"); commandHelp.put("registers", "Shows or sets CPU register values.\nUsage: registers [reg value]\nExample: registers PC 4000"); - commandHelp.put("break", "Manages execution breakpoints.\nUsage: break | break - | break clear | break list"); + commandHelp.put("break", "Manages execution breakpoints.\nUsage: break - List all active breakpoints\n break - Add a breakpoint\n break - - Remove a breakpoint\n break clear - Remove all breakpoints"); commandHelp.put("runto", "Runs the CPU until it reaches the specified address.\nUsage: runto (or rt )"); commandHelp.put("run", "Runs the CPU for a specified number of cycles or until a breakpoint is hit.\n" + diff --git a/src/main/java/jace/terminal/MonitorMode.java b/src/main/java/jace/terminal/MonitorMode.java index 60cb4d5..a546978 100644 --- a/src/main/java/jace/terminal/MonitorMode.java +++ b/src/main/java/jace/terminal/MonitorMode.java @@ -315,7 +315,8 @@ public class MonitorMode implements TerminalMode, WatchCallback { " reg PC $C000 - Set program counter to $C000\n" + " reg X 42 - Set X register to decimal 42"); commandHelp.put("break", "Manages breakpoints.\n" + - "Usage: break addr - Add a breakpoint at address\n" + + "Usage: break - List all active breakpoints\n" + + " break addr - Add a breakpoint at address\n" + " break -addr - Remove a breakpoint at address\n" + " break clear - Remove all breakpoints"); commandHelp.put("step", "Steps through CPU instructions.\n" + @@ -459,7 +460,7 @@ public class MonitorMode implements TerminalMode, WatchCallback { void handleBreakpoint(String[] args) { if (args.length == 0) { - output.println("Usage: break
or break -
or break clear"); + listBreakpoints(); return; }