Review Request 123900: GDB plugin enhancements and some command queue optimizations for master branch

Andreas Roth aroth at arsoft-online.com
Sun Jun 7 13:26:40 UTC 2015


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

(Updated June 7, 2015, 1:26 p.m.)


Review request for KDevelop.


Changes
-------

keep original GDB (exec) command order to fix gdb unit tests


Repository: kdevelop


Description
-------

Basically this contains the same changes as https://git.reviewboard.kde.org/r/121456/ but for the master branch.

Repo is available at https://github.com/aroth-arsoft/kdevelop/tree/master-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/debugsession.cpp fe015ae915f190cef266f1c52d0cedf16dfb9007 
  debuggers/gdb/disassemblewidget.h c49e5bc1d9a9234888006e91127a880ef257ba23 
  debuggers/gdb/disassemblewidget.cpp d1735e0ced0523519d2f3864c018bf1d6cfe8f7f 
  debuggers/gdb/gdb.cpp 66fee9e2f5f44a2f76564d9df5f701689a27a874 
  debuggers/gdb/gdbcommand.h 692c0ac6b6a98b71678f2aa89b8f7940ed390a2e 
  debuggers/gdb/gdbcommand.cpp 3207fdc896ec0dd0f7de319547bf4302042872a4 
  debuggers/gdb/gdbcommandqueue.h 9525301bf0614d7613d58ae4dfdcc08da709f7b4 
  debuggers/gdb/gdbcommandqueue.cpp 4bf11d0bd68e1aa0ef1cf2a8008011cf0fb76a55 
  debuggers/gdb/memviewdlg.cpp 08034e6631fc8f103005471a08cf0df031ff85f3 

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


Testing (updated)
-------

run gdb unit tests


Thanks,

Andreas Roth

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


More information about the KDevelop-devel mailing list