Review Request 121456: GDB plugin enhancements and some command queue optimizations

[Andreas Roth]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121456/
-----------------------------------------------------------

(Updated Dec. 13, 2014, 12:33 p.m.)


Review request for KDevelop.


Repository: kdevelop


Description
-------

Repo is available at https://github.com/aroth-arsoft/kdevelop/tree/4.7-gdb-enhancements
- switch the assembler flavor from AT&T to Intel and back in the dissassembler widget
- remove StackList commands from the queue when a new Exec command arrives. This reduces the wait time when stepping through the code quickly.
- remove duplicated command in the queue. Mostly to eliminate duplicate variable updates.
- show the pending command in the command queue. This is purely for debugging and to check which commands are currently pending.
- slight improvements to memory view to use sizeof() when no size argument is given


Diffs (updated)
-----

  debuggers/gdb/gdbcommandqueue.cpp 801bc41e79f62a226de04cd1a91500c86a9cf1a3 
  debuggers/gdb/memviewdlg.cpp cf9d0b49ea0b617b405a5065354af9c9e59afa8d 
  debuggers/gdb/gdbcommandqueue.h 7d82f45a244d9fffb9b3a2ba5a82eaaa06f5c8d0 
  debuggers/gdb/gdbcommand.cpp 67abbd87f1f0d10a84a45b3ee9f104c77ffe925b 
  debuggers/gdb/disassemblewidget.h ccffc2f0325aac8c8486ff413ed64219548461c3 
  debuggers/gdb/disassemblewidget.cpp 91194ed7e69ff87ebdfbb5383c1b38939760d098 
  debuggers/gdb/gdb.cpp 46420876149604a318caec7111f252dcc12b39b4 
  debuggers/gdb/gdbcommand.h e1ebc6988fb508a187d2775735c13b9123fcda04 
  debuggers/gdb/debugsession.cpp edc0d34473e99230db510457fd609a178bc8fa34 

Diff: https://git.reviewboard.kde.org/r/121456/diff/


Testing
-------

Only preliminary testing done; haven't checked the unit-tests yets


Thanks,

Andreas Roth

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20141213/c30a13ed/attachment.html>