Review Request 122898: port kdev-control-graph to kf5

[Martin Tobias Holmedahl Sandsmark]

> On March 11, 2015, 1:38 a.m., Aleix Pol Gonzalez wrote:
> > CMakeLists.txt, line 35
> > <https://git.reviewboard.kde.org/r/122898/diff/1/?file=354206#file354206line35>
> >
> >     Please fix it then. Also worksforme, I think. Can you maybe point out what's wrong actually?

/usr/include/kdevplatform/language/util/navigationtooltip.h:26:10: fatal error: 'languageexport.h' file not found

is what I get without it.

changing it to #include <language/languageexport.h> works for me, but I'm not sure if it is correct?

I removed it anyways, so I have to fix it before committing this. :-)


> On March 11, 2015, 1:38 a.m., Aleix Pol Gonzalez wrote:
> > duchaincontrolflow.cpp, line 375
> > <https://git.reviewboard.kde.org/r/122898/diff/1/?file=354212#file354212line375>
> >
> >     IIRC, there's a .url() method you can use there. This way you don't need to construct.
> >     
> >     In fact, it's wrong, otherwise you should use QUrl(). .str() will return already a url, no need to do the ::fromLocalFile (which prefixes the file://)

toUrl(), don't know how I missed that.


- Martin Tobias Holmedahl


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


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/20150311/4ecbd06c/attachment.html>