Review Request 112039: Add a CPU registers view.

Vlas Puhov vlas.puhov at mail.ru
Sat Aug 24 07:46:16 UTC 2013


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

(Updated Aug. 24, 2013, 11:46 a.m.)


Review request for KDevelop.


Changes
-------

*Register names now static.
*Removed a lot of debug massages.
*New registerNamesForGroup algorithm.
...


Description
-------

Supported architectures: x86/x86_64, arm v7(v6 should work too).
Features: x86/x86_64: General registers/flags, segment, FPU registers read/write. XMM - read only.
arm: General registers/flags, VFP single - read/write, other - read only.

Related changes:
DisassembleWidget: removed startAddress and endAddress combo boxes so as to let to resize disassemble view, but the change address functionality is still there through context menu.
FramestackModel: if first frame doesn't contain debug information don't set currentPosition to the first frame with debug information. It allows to keep synchronization between views(gdb, editor, disassemble widget and registers) otherwise all these views'll be in different states.


This addresses bug 273152.
    http://bugs.kde.org/show_bug.cgi?id=273152


Diffs (updated)
-----

  debuggers/gdb/registers/registersview.ui PRE-CREATION 
  debuggers/gdb/registers/registersview.cpp PRE-CREATION 
  debuggers/gdb/registers/registersview.h PRE-CREATION 
  debuggers/gdb/registers/registersmanager.cpp PRE-CREATION 
  debuggers/gdb/registers/registersmanager.h PRE-CREATION 
  debuggers/gdb/registers/registercontroller_x86.cpp PRE-CREATION 
  debuggers/gdb/registers/registercontroller_arm.cpp PRE-CREATION 
  debuggers/gdb/registers/registercontroller_x86.h PRE-CREATION 
  debuggers/gdb/registers/registercontroller_arm.h PRE-CREATION 
  debuggers/gdb/registers/registercontroller.cpp PRE-CREATION 
  debuggers/gdb/registers/registercontroller.h PRE-CREATION 
  debuggers/gdb/disassemblewidget.cpp e1d6e8f 
  debuggers/gdb/disassemblewidget.h 81c25fc 
  debuggers/gdb/debuggerplugin.cpp 3533cdb 
  debuggers/gdb/CMakeLists.txt 32ef14f 

Diff: http://git.reviewboard.kde.org/r/112039/diff/


Testing
-------

Manual only for x86/x86_64 and armv7 architectures 


File Attachments (updated)
----------------

Related changes in kdevplatform
  http://git.reviewboard.kde.org/media/uploaded/files/2013/08/12/framestackmodel.diff
The GUI screenshots
  http://git.reviewboard.kde.org/media/uploaded/files/2013/08/23/GUI_screenshots.7z


Thanks,

Vlas Puhov

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


More information about the KDevelop-devel mailing list