Review Request 121616: IDebugSession: reduce coupling between interface class and implementations

Nicolai Hähnle nhaehnle at gmail.com
Mon Dec 22 09:11:17 UTC 2014


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

(Updated Dec. 22, 2014, 9:11 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDevelop.


Repository: kdevplatform


Description
-------

Instead of having the derived class poke at IDebugSession member variables,
we now use pure virtual methods to access IBreakpointController etc.
This is in line with the design of other interfaces throughout kdevplatform.

Note that this change is both forward and backward source incompatible, i.e.
debugger plugins must change simultaneously with this change or they will
no longer compile. Since this change is fairly trivial, this seems acceptable.


Diffs
-----

  debugger/interfaces/idebugsession.cpp 193701d8b87de1d7712bb6b091e42ddee4cc0103 
  debugger/interfaces/idebugsession.h 219c94865e76fc5e70247f6b96add6eb42391ff5 

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


Testing
-------

manual testing


Thanks,

Nicolai Hähnle

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20141222/2a82012d/attachment-0001.html>


More information about the KDevelop-devel mailing list