Review Request 122898: port kdev-control-graph to kf5
Kevin Funk
kfunk at kde.org
Thu Apr 9 21:00:58 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122898/#review78724
-----------------------------------------------------------
Ship it!
duchaincontrolflowinternaljob.h (line 32)
<https://git.reviewboard.kde.org/r/122898/#comment53865>
Remove superfluos WS (3x)
duchaincontrolflowinternaljob.h (line 45)
<https://git.reviewboard.kde.org/r/122898/#comment53866>
Use 'override' keyword
LGTM
- Kevin Funk
On March 11, 2015, 2:03 a.m., Martin Tobias Holmedahl Sandsmark wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122898/
> -----------------------------------------------------------
>
> (Updated March 11, 2015, 2:03 a.m.)
>
>
> Review request for KDevelop and Sandro Andrade.
>
>
> Repository: kdev-control-flow-graph
>
>
> Description
> -------
>
> quick and dirty port
>
>
> Diffs
> -----
>
> CMakeLists.txt bf65ed3
> controlflowgraphfiledialog.h 51217d6
> controlflowgraphfiledialog.cpp f790efd
> controlflowgraphnavigationcontext.cpp 8bf1d21
> controlflowgraphview.cpp 0a9ba01
> duchaincontrolflow.h 69d5eec
> duchaincontrolflow.cpp d94f55d
> duchaincontrolflowinternaljob.h ffb6da5
> duchaincontrolflowinternaljob.cpp 67132a5
> duchaincontrolflowjob.h 756c600
> duchaincontrolflowjob.cpp 158e6f6
> kdevcontrolflowgraphview.desktop.cmake 27b94d0
> kdevcontrolflowgraphview.json PRE-CREATION
> kdevcontrolflowgraphviewplugin.cpp 44cc60d
>
> Diff: https://git.reviewboard.kde.org/r/122898/diff/
>
>
> Testing
> -------
>
> used it, together with the kgraphview port in the other review request.
>
>
> Thanks,
>
> Martin Tobias Holmedahl Sandsmark
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150409/95919fe0/attachment.html>
More information about the KDevelop-devel
mailing list