[Github-comments] [geany-plugins] debugger: Various stuff (#349)

[Colomban Wendling]

All-in-one PR updating the debugger plugin.  Even if it's a large PR in itself, it is properly split: each commit (should be) relevant on its own, and there are merges from internal feature branches in the way that create bigger steps.

Highlights:
* Very noticeable speedup showing a large stack trace (fixes #347).  With enormous stack traces building the list went from >100s to <0.3s (297be8d, and additional less relevant ones).  It's still actually slower than that, but apparently it's GDB who's taking like 10s to pass in the whole list.  Anyway, it's far more usable now.
* Add basic thread switching support (44c8d36, ca9abb6).
* Fix handling of files with escape sequences in their name, like `foo "bar` or `foo\bar` (part of #83) (d21c653).
* Fix displaying the current instruction marker when there is another frame at the same location (186d3fa).
* Report the signal name and meaning when the debugged process stops because of a signal (dc049f4).
* Fixes various not-too-likely crasher bugs (1321161, df08535, a52777f).
* Plugs a few memory leaks.
* Small code cleanups.

PS: no, I'm still not the real maintainer of this orphaned plugin, but I started looking at it because of #347 and went down trying stuff and fixing others on the way.
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany-plugins/pull/349

-- Commit Summary --

  * debugger: Fix tooltips in the Call Stack
  * debugger: Speed up building several lists
  * debugger: Fix missing sentinel in gtk_tree_model_get() call
  * debugger: Add reference counting to frame objects
  * debugger: stree: Cleanup stored columns
  * debugger: Greatly improve Stack Tree population speed
  * debugger/gdb_mi: Improve implementation of parse_cstring()
  * debugger/gdb_mi: Improve implementation of parse_string()
  * debugger/gdb_mi: Do not allocate values when reading invalid input
  * debugger/gdb_mi: Test: allow for lines of arbitrary length
  * Merge branch 'debugger/gdb_mi-speedup' into debugger/frame-speedup
  * debugger: stree: Don't maintain a table of thread row references
  * Merge branch 'debugger/frame-speedup' into debugger/master
  * debugger: Avoid potential NULL dereference
  * debugger: Fix a few memory leaks
  * debugger: Remove some unnecessary uses of GString wrappers
  * debugger: Report the signal name when the process received one
  * debugger: Use an enumeration for markers
  * debugger: Fix marker for current frame
  * debugger: Remove an unused and incorrect macro
  * debugger/gdb_mi: Slightly reduce memory usage
  * debugger: Use tree view cursor instead of selection in stack tree
  * debugger: Avoid possible NULL dereference
  * debugger: Add debugger module-level support for switching threads
  * debugger: Allow to switch active thread
  * debugger: Fix handling of escape characters in file names
  * Merge branch 'debugger/thread-support' into debugger/master

-- File Changes --

    M debugger/src/breakpoints.c (6)
    M debugger/src/breakpoints.h (1)
    M debugger/src/dbm_gdb.c (216)
    M debugger/src/debug.c (71)
    M debugger/src/debug_module.c (28)
    M debugger/src/debug_module.h (13)
    M debugger/src/envtree.c (6)
    M debugger/src/gdb_mi.c (71)
    M debugger/src/gdb_mi.h (6)
    M debugger/src/markers.c (19)
    M debugger/src/stree.c (357)
    M debugger/src/stree.h (4)
    M debugger/src/watch_model.c (4)

-- Patch Links --

https://github.com/geany/geany-plugins/pull/349.patch
https://github.com/geany/geany-plugins/pull/349.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/349
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160208/b027ad3a/attachment.html>