Review Request 118313: [Konqlib] KUrl -> QUrl

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Sun May 25 13:31:43 UTC 2014


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



lib/konq/konq_popupmenu.cpp
<https://git.reviewboard.kde.org/r/118313/#comment40624>

    KUrl::cleanPath() -> QUrl::???



lib/konq/konq_popupmenu.cpp
<https://git.reviewboard.kde.org/r/118313/#comment40625>

    KUrl::cleanPath() -> QUrl::???



lib/konq/konqmimedata.cpp
<https://git.reviewboard.kde.org/r/118313/#comment40626>

    Is "mimeData->setUrls(kdeUrls);" correct?



lib/konq/konqmimedata.cpp
<https://git.reviewboard.kde.org/r/118313/#comment40627>

    Is "mimeData->setUrls(mostLocalURLs);" correct?


- Emmanuel Pescosta


On May 25, 2014, 3:27 p.m., Emmanuel Pescosta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118313/
> -----------------------------------------------------------
> 
> (Updated May 25, 2014, 3:27 p.m.)
> 
> 
> Review request for KDE Base Apps, KDE Frameworks and David Faure.
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> Ported Konqlib from KUrl to QUrl (convert-kurl.pl)
> 
> Still some ToDos left in the code (I'll mark it inline)
> 
> 
> Diffs
> -----
> 
>   lib/konq/favicons/favicons.h 30f0272 
>   lib/konq/favicons/favicons.cpp 4646fa5 
>   lib/konq/konq_copytomenu.h 0958e20 
>   lib/konq/konq_copytomenu.cpp 7ee5131 
>   lib/konq/konq_copytomenu_p.h eb6649a 
>   lib/konq/konq_dndpopupmenuplugin.h 2417080 
>   lib/konq/konq_historyentry.h 6b33b98 
>   lib/konq/konq_historyentry.cpp bfc4a5d 
>   lib/konq/konq_historyprovider.h 91b5b87 
>   lib/konq/konq_historyprovider.cpp 10a5e74 
>   lib/konq/konq_nameandurlinputdialog.h 3a943f7 
>   lib/konq/konq_nameandurlinputdialog.cpp 2fb06f9 
>   lib/konq/konq_operations.h b51fa3f 
>   lib/konq/konq_operations.cpp 0cbff08 
>   lib/konq/konq_popupmenu.h b5f5e39 
>   lib/konq/konq_popupmenu.cpp caa37e7 
>   lib/konq/konq_popupmenuinformation.h 1454073 
>   lib/konq/konq_popupmenuinformation.cpp 6e1033b 
>   lib/konq/konq_sound.h 8063073 
>   lib/konq/konq_sound.cc f8a80a9 
>   lib/konq/konqmimedata.h a2640a2 
>   lib/konq/konqmimedata.cpp 8f9ecf1 
>   lib/konq/tests/favicontest.cpp d48b03d 
>   lib/konq/tests/konqmimedatatest.cpp 6a2be4f 
>   lib/konq/tests/konqpopupmenutest.cpp 46851c5 
> 
> Diff: https://git.reviewboard.kde.org/r/118313/diff/
> 
> 
> Testing
> -------
> 
> Tests fail
> 
> konqmimedatatest: (todos)
> FAIL!  : KonqMimeDataTest::testPopulate() 'mimeData->hasUrls()' returned FALSE.
> FAIL!  : KonqMimeDataTest::testCut() 'mimeData->hasUrls()' returned FALSE.
> 
> konqpopupmenutest: (local setup problem maybe?!)
> QFATAL : KonqPopupMenuTest::testFile() ASSERT failure in KServiceTypeFactory::KServiceTypeFactory()
> 
> favicontest: (local setup problem maybe?!)
> QFATAL : FavIconTest::testSetIconForURL() ASSERT failure in KServiceTypeFactory::KServiceTypeFactory()
> 
> 
> Thanks,
> 
> Emmanuel Pescosta
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140525/c3e58a75/attachment.html>


More information about the Kde-frameworks-devel mailing list