Review Request 122905: Update kdev-valgrind to KF5 compatibility

[Laszlo Kis-Adam]

> On March 13, 2015, 12:54 a.m., Aleix Pol Gonzalez wrote:
> > models/memcheckitemsimpl.cpp, line 208
> > <https://git.reviewboard.kde.org/r/122905/diff/2/?file=354710#file354710line208>
> >
> >     That's still not correct. Instead do the following:
> >     
> >     QUrl base = QUrl::fromLocalFile(QDir::cleanPath(dir+'/'));
> >     QUrl url = base.resolved(QUrl(file));
> >     
> >     Also you could only do the resolution if QUrl(file).isRelative()

It is always relative as the code provides just the filename in the file variable, so it's pointless to check if it's relative.
Also the code works, valgrind provides the data in the format of directory and filename. The directory is always without trailing slash, and the filename is always just a filename.

Here's an example of what memcheck provides, that is then parsed:
      <dir>/home/dfighter/projects/kdevtest/my/super/deep/example/directory</dir>
      <file>tests.cpp</file>

As for the other valgrind tools, if they don't provide the right directory ( massif doesn't those items always have the directory set to the base directory ) the resolution will NOT work either way. So nothing lost, nothing gained.

So what is incorrect exactly?
Are you just being pedantic, or is there an actual use case where this is incorrect? If there is, ofc I will change it :)
Thanks!


- Laszlo


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


On March 13, 2015, 12:02 a.m., Laszlo Kis-Adam wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122905/
> -----------------------------------------------------------
> 
> (Updated March 13, 2015, 12:02 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdev-valgrind
> 
> 
> Description
> -------
> 
> Updated kdev-valgrind to be compatible with KF5
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 31aaf52 
>   cmake/FindKDevPlatform.cmake 1a771c5 
>   config.h a7ec923 
>   config.cpp 9924dca 
>   config/cachegrindconfigpage.h 070fc06 
>   config/cachegrindconfigpage.cpp a487cc5 
>   config/callgrindconfigpage.h eefbb18 
>   config/callgrindconfigpage.cpp 2c97477 
>   config/genericconfigpage.h e6c9108 
>   config/genericconfigpage.cpp d0147f9 
>   config/helgrindconfigpage.h 497e328 
>   config/helgrindconfigpage.cpp 3fd97bd 
>   config/massifconfigpage.h 64f0f11 
>   config/massifconfigpage.cpp 49660af 
>   config/memcheckconfigpage.h f1d5925 
>   config/memcheckconfigpage.cpp 3a661db 
>   config/ui/cachegrindconfig.ui 170731e 
>   config/ui/callgrindconfig.ui fbb003d 
>   config/ui/genericconfig.ui a09f301 
>   config/ui/helgrindconfig.ui 6cead3c 
>   config/ui/massifconfig.ui 6d4a8de 
>   config/ui/memcheckconfig.ui 11a163e 
>   config/valgrindcachegrindconfigpage.h 73f65de 
>   config/valgrindcachegrindconfigpage.cpp 22b349b 
>   config/valgrindcallgrindconfigpage.h fbdc90d 
>   config/valgrindcallgrindconfigpage.cpp 5820af3 
>   config/valgrindgenericconfigpage.h cdaf0a8 
>   config/valgrindgenericconfigpage.cpp e05ef52 
>   config/valgrindhelgrindconfigpage.h c7487c2 
>   config/valgrindhelgrindconfigpage.cpp b6e877d 
>   config/valgrindmassifconfigpage.h 62e3e79 
>   config/valgrindmassifconfigpage.cpp 9165f6e 
>   config/valgrindmemcheckconfigpage.h 431040a 
>   config/valgrindmemcheckconfigpage.cpp 7c4f92f 
>   debug.h PRE-CREATION 
>   debug.cpp PRE-CREATION 
>   job.h ec4b1b0 
>   job.cpp cf832c4 
>   jobs/cachegrindjob.cpp 0028a62 
>   jobs/callgrindjob.cpp 627e4f7 
>   jobs/massifjob.cpp 9fe2fde 
>   jobs/memcheckjob.cpp 6912760 
>   kdevvalgrind.desktop fb76bb6 
>   kdevvalgrind.desktop.cmake PRE-CREATION 
>   launcher.h 539e34f 
>   launcher.cpp d9b52af 
>   marks.cpp ff2c890 
>   models/cachegrinditem.h 311d228 
>   models/cachegrinditem.cpp 1b50b63 
>   models/cachegrindmodel.cpp 70cad22 
>   models/callgrinditem.h 11c4a33 
>   models/callgrinditem.cpp 845104b 
>   models/callgrindmodel.cpp 5dc971c 
>   models/massifitem.h 6460fba 
>   models/massifitem.cpp 33cd92d 
>   models/massifmodel.cpp 4d5f66a 
>   models/memcheckitemsimpl.h 111e4a6 
>   models/memcheckitemsimpl.cpp fdee8d2 
>   models/memcheckmodel.cpp 2b96fbe 
>   parsers/cachegrindparser.h 312f09a 
>   parsers/cachegrindparser.cpp f82fd8a 
>   parsers/callgrindparser.h e7f522d 
>   parsers/callgrindparser.cpp 5ffe4c9 
>   parsers/massifparser.h 1a97f3b 
>   parsers/memcheckparser.h 2e3d5ae 
>   parsers/memcheckparser.cpp cf438ea 
>   plugin.h e0410e4 
>   plugin.cpp 3cd2448 
>   statjob.h PRE-CREATION 
>   statjob.cpp PRE-CREATION 
>   views/cachegrindview.cpp 591f217 
>   views/callgrindview.cpp 6aa551d 
>   views/massifview.cpp 7bb275f 
>   views/memcheckview.cpp 95961fb 
>   widget.cpp 18eacf7 
> 
> Diff: https://git.reviewboard.kde.org/r/122905/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Laszlo Kis-Adam
> 
>

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