Book: update for firmware version 1.8 and hardware version 2.3.

BMDebug: autocompletion for help & info commands.
BMFlash: added copyright in help/about.
Nuklear: respond to button-release of the mouse button, rather than on mouse press.

Author: compuphase

BlackMagicProbe.pdf

@@ -807,14 +807,14 @@ static int console_autocomplete(char *text, size_t textsize, const DWARF_SYMBOLL
     { "down", NULL, NULL },
     { "enable", NULL, NULL },
     { "file", NULL, "%path" },
-    { "find", NULL, NULL },
+    { "find", NULL, "%func %var" },
     { "finish", "fin", NULL },
     { "frame", "f", NULL },
-    { "help", NULL, "break breakpoints data files keyboard monitor running serial stack status support svd trace user-defined" },
+    { "help", NULL, "assembly break breakpoints data files find keyboard monitor running serial stack status support svd trace user-defined" },
     { "info", NULL, "args breakpoints frame functions locals scope set sources stack svd variables vtbl" },
     { "list", NULL, "%func %var %file" },
     { "load", NULL, NULL },
-    { "monitor", "mon", "connect_srst hard_srst jtag_scan morse option swdp_scan targets tpwr traceswo vector_catch" },
+    { "monitor", "mon", "connect_srst frequency halt_timeout hard_srst heapinfo jtag_scan morse option swdp_scan targets tpwr traceswo vector_catch" },
     { "next", "n", NULL },
     { "print", "p", "%var %reg" },
     { "ptype", NULL, "%var" },
@@ -4055,6 +4055,10 @@ static bool handle_help_cmd(char *command, STRINGLIST *textroot, int *active,
     *active = POPUP_HELP;
     *reformat = nk_true;    /* by default, reformat (overruled for "help mon") */
     if (*cmdptr == '\0') {
+      stringlist_append(textroot, "BMDebug is a GDB front-end, specifcally for embedded debugging with the Black Magic Probe.", 0);
+      stringlist_append(textroot, "Copyright 2019-2022 CompuPhase", 0);
+      stringlist_append(textroot, "Licensed under the Apache License version 2.0", 0);
+      stringlist_append(textroot, "", 0);
       stringlist_append(textroot, "Front-end topics.", 0);
       stringlist_append(textroot, "", 0);
       stringlist_append(textroot, "assembly -- show assembly mixed with source code.", 0);
@@ -4124,6 +4128,7 @@ static bool handle_help_cmd(char *command, STRINGLIST *textroot, int *active,
       stringlist_append(textroot, "", 0);
       stringlist_append(textroot, "reset -- restart the target program; keep breakpoints and variable watches.", 0);
       stringlist_append(textroot, "reset hard -- restart both the debugger and the target program.", 0);
+      stringlist_append(textroot, "reset load -- restart the debugger and reload the target program.", 0);
       return true;
     } else if (TERM_EQU(cmdptr, "serial", 6)) {
       stringlist_append(textroot, "Configure the serial monitor.", 0);
@@ -5314,6 +5319,10 @@ static void help_popup(struct nk_context *ctx, APPSTATE *state, float canvas_wid
       nk_group_set_scroll(ctx, "help", xoffs, yoffs);
     }
 
+    /* other keyboard functions */
+    if (nk_input_is_key_pressed(&ctx->input, NK_KEY_TAB))
+      console_autocomplete(state->help_edit, sizearray(state->help_edit), &dwarf_symboltable);
+
     nk_popup_end(ctx);
   } else {
     state->popup_active = POPUP_NONE;

doc/BlackMagicProbe.odt

@@ -639,6 +639,8 @@ static int help_popup(struct nk_context *ctx)
     "Black Magic Probe. It automatically handles idiosyncrasies of MCU\n"
     "families, and supports setting a serial number during the download\n"
     "(i.e. serialization). It does not require GDB.\n\n"
+    "Copyright 2019-2022 CompuPhase\n"
+    "Licensed under the Apache License version 2.0\n\n"
     "^3Options\n"
     "The MCU family must be set for post-processing the ELF file or\n"
     "performing additional configurations before the download. It is\n"

doc/bmp-v23-top.jpg

@@ -11997,6 +11997,12 @@ nk_input_button(struct nk_context *ctx, enum nk_buttons id, int x, int y, nk_boo
     btn->clicked_pos.y = (float)y;
     btn->down = down;
     btn->clicked++;
+    #ifdef NK_BUTTON_TRIGGER_ON_RELEASE
+        if (down == 1 && id == NK_BUTTON_LEFT) {
+            in->mouse.down_pos.x = btn->clicked_pos.x;
+            in->mouse.down_pos.y = btn->clicked_pos.y;
+        }
+    #endif
 }
 NK_API void
 nk_input_scroll(struct nk_context *ctx, struct nk_vec2 val)
@@ -12057,8 +12063,14 @@ nk_input_has_mouse_click_in_rect(const struct nk_input *i, enum nk_buttons id,
     const struct nk_mouse_button *btn;
     if (!i) return nk_false;
     btn = &i->mouse.buttons[id];
-    if (!NK_INBOX(btn->clicked_pos.x,btn->clicked_pos.y,b.x,b.y,b.w,b.h))
-        return nk_false;
+    #ifdef NK_BUTTON_TRIGGER_ON_RELEASE
+        if (!NK_INBOX(btn->clicked_pos.x,btn->clicked_pos.y,b.x,b.y,b.w,b.h)
+            || !NK_INBOX(i->mouse.down_pos.x,i->mouse.down_pos.y,b.x,b.y,b.w,b.h))
+            return nk_false;
+    #else
+        if (!NK_INBOX(btn->clicked_pos.x,btn->clicked_pos.y,b.x,b.y,b.w,b.h))
+            return nk_false;
+    #endif
     return nk_true;
 }
 NK_API nk_bool
@@ -21513,7 +21525,8 @@ nk_do_edit(nk_flags *state, struct nk_command_buffer *out,
         }
 
         /* draw text */
-        {struct nk_color background_color;
+        { /* local block */
+        struct nk_color background_color;
         struct nk_color text_color;
         struct nk_color sel_background_color;
         struct nk_color sel_text_color;
@@ -21632,7 +21645,8 @@ nk_do_edit(nk_flags *state, struct nk_command_buffer *out,
                 nk_fill_rect(out, label, 0, cursor_color);
                 nk_widget_text(out, label, cursor_ptr, glyph_len, &txt, NK_TEXT_LEFT, font);
             }
-        }}
+        }
+        } /* end local */
     } else {
         /* not active so just draw text */
         int l = nk_str_len_char(&edit->string);
@@ -21792,14 +21806,18 @@ nk_edit_buffer(struct nk_context *ctx, nk_flags flags,
         if (flags & NK_EDIT_CLIPBOARD)
             edit->clip = ctx->clip;
         edit->active = (unsigned char)win->edit.active;
-    } else edit->active = nk_false;
+    } else {
+        edit->active = nk_false;
+    }
+    if (nk_is_popup_open(ctx))
+        edit->active = nk_false;
     edit->mode = win->edit.mode;
 
     filter = (!filter) ? nk_filter_default: filter;
     prev_state = (unsigned char)edit->active;
     in = (flags & NK_EDIT_READ_ONLY) ? 0: in;
     ret_flags = nk_do_edit(&ctx->last_widget_state, &win->buffer, bounds, flags,
-                    filter, edit, &style->edit, in, style->font);
+                           filter, edit, &style->edit, in, style->font);
 
     if (ctx->last_widget_state & NK_WIDGET_STATE_HOVER)
         ctx->style.cursor_active = ctx->style.cursors[NK_CURSOR_TEXT];
@@ -23797,9 +23815,9 @@ nk_is_popup_open(struct nk_context *ctx)
     struct nk_window *popup;
 
     NK_ASSERT(ctx != NULL);
-    win = ctx->current;
     if (ctx->current != ctx->active)
         return nk_false;
+    win = ctx->current;
     popup = win->popup.win;
     return (popup != NULL);
 }