Review Request 112039: Add a CPU registers view.

Vlas Puhov vlas.puhov at mail.ru
Tue Aug 20 18:59:31 UTC 2013


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

(Updated Aug. 20, 2013, 10:59 p.m.)


Review request for KDevelop.


Changes
-------

*Further const'ness of parameters/variables.
*Early-return statement is used.
*Reformatting with kdevelop's KDELibs formatting style.
...


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/CMakeLists.txt 32ef14f 
  debuggers/gdb/debuggerplugin.cpp 3533cdb 
  debuggers/gdb/disassemblewidget.h 81c25fc 
  debuggers/gdb/disassemblewidget.cpp e1d6e8f 
  debuggers/gdb/registers/registercontroller.h PRE-CREATION 
  debuggers/gdb/registers/registercontroller.cpp PRE-CREATION 
  debuggers/gdb/registers/registercontroller_arm.h PRE-CREATION 
  debuggers/gdb/registers/registercontroller_arm.cpp PRE-CREATION 
  debuggers/gdb/registers/registercontroller_x86.h PRE-CREATION 
  debuggers/gdb/registers/registercontroller_x86.cpp PRE-CREATION 
  debuggers/gdb/registers/registersmanager.h PRE-CREATION 
  debuggers/gdb/registers/registersmanager.cpp PRE-CREATION 
  debuggers/gdb/registers/registersview.h PRE-CREATION 
  debuggers/gdb/registers/registersview.cpp PRE-CREATION 
  debuggers/gdb/registers/registersview.ui PRE-CREATION 

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


Testing
-------

Manual only for x86/x86_64 and armv7 architectures 


File Attachments
----------------

Related changes in kdevplatform
  http://git.reviewboard.kde.org/media/uploaded/files/2013/08/12/framestackmodel.diff


Thanks,

Vlas Puhov

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


More information about the KDevelop-devel mailing list