I've noticed that mi_cmd_break_insert does some fairly contrived logic which amounts to passing either 0 or 1 as the value of 'hardware' parameter to set_breakpoint. Further, set_breakpoint appears to be trivial and useless wrapper over break_command_really. This patch cleans that up. Are breakpoint.{c,h} changes OK? - Volodya