Review Request: Common Debugger infrastructure

Niko Sams niko.sams at gmail.com
Sat Mar 21 21:46:23 UTC 2009


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

Review request for KDevelop.


Summary
-------

- IDebugController: has just one slot to add a Debug-Session, debuggers/shell implements this interface.
  Debugger-plugins search for the debuggers/shell plugin and call IDebugController::addSession when
  a debug session is started.
- IDebugSession: debugger plugin inherits from this class to use the debugger UI and communicate
  with the debuggers/shell plugin.

Currenly only the various actions are implemented; Breakpoints, Variable-Views, Stack-Views etc.
should be possible in a similar way.

Please review my ideas and tell me if I go into a wrong direction - or what could be improved.


Diffs
-----

  trunk/KDE/kdevplatform/CMakeLists.txt 941839 
  trunk/KDE/kdevplatform/KDevPlatformConfig.cmake.in 941839 
  trunk/KDE/kdevplatform/cmake/modules/FindKDevPlatform.cmake 941839 
  trunk/KDE/kdevplatform/debugger/CMakeLists.txt PRE-CREATION 
  trunk/KDE/kdevplatform/debugger/debuggerexport.h PRE-CREATION 
  trunk/KDE/kdevplatform/debugger/interfaces/idebugsession.h PRE-CREATION 
  trunk/KDE/kdevplatform/debugger/interfaces/idebugsession.cpp PRE-CREATION 
  trunk/KDE/kdevplatform/debugger/shell/CMakeLists.txt PRE-CREATION 
  trunk/KDE/kdevplatform/debugger/shell/debuggershellplugin.h PRE-CREATION 
  trunk/KDE/kdevplatform/debugger/shell/debuggershellplugin.cpp PRE-CREATION 
  trunk/KDE/kdevplatform/debugger/shell/kdevdebuggershell.desktop PRE-CREATION 
  trunk/KDE/kdevplatform/debugger/shell/kdevdebuggershellui.rc PRE-CREATION 
  trunk/KDE/kdevplatform/interfaces/CMakeLists.txt 941839 
  trunk/KDE/kdevplatform/interfaces/idebugcontroller.h PRE-CREATION 
  trunk/KDE/kdevplatform/interfaces/idebugcontroller.cpp PRE-CREATION 

Diff: http://reviewboard.kde.org/r/379/diff


Testing
-------

ported the xdebug plugin


Thanks,

Niko





More information about the KDevelop-devel mailing list