Review Request 128845: [ktp-common-internals] [debugger] Cleanup and reformat

Alexandr Akulich akulichalexander at gmail.com
Tue Sep 6 12:11:16 UTC 2016


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

(Updated Sept. 6, 2016, 12:11 p.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy.


Changes
-------

Submitted with commit 17b0f5e4a7600487b9d92e766c6f1f44b8aa014a by Alexandr Akulich to branch master.


Repository: ktp-common-internals


Description
-------

Removed unneeded includes, destructor moved to be the next after constructor, removed incorrect "virtual" usage, fixed braces and so on.

This is a preparation to logic and ui split up, which I can not upload until this commit will be landed.


Diffs
-----

  tools/debugger/debug-message-view.h 0bde707 
  tools/debugger/debug-message-view.cpp af09715 
  tools/debugger/main-window.h 2e7c968 

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


Testing
-------

Compiles


Thanks,

Alexandr Akulich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20160906/b53321cf/attachment.html>


More information about the KDE-Telepathy mailing list