Review Request 125069: [patch] Port dolphin-plugins to KF5 without kdelibs4support

[Emmanuel Pescosta]

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



git/checkoutdialog.cpp (line 43)
<https://git.reviewboard.kde.org/r/125069/#comment59347>

    Please remove all whitespaces which are marked as red blocks



git/fileviewgitplugin.cpp (lines 57 - 90)
<https://git.reviewboard.kde.org/r/125069/#comment59344>

    <application></application> was correct, not need to remove it. (applies to all such changes in this patch)



svn/fileviewsvnplugin.cpp (line 323)
<https://git.reviewboard.kde.org/r/125069/#comment59346>

    KWindowConfig::restoreWindowSize(dialog.windowHandle(), dialogConfig);



svn/fileviewsvnplugin.cpp 
<https://git.reviewboard.kde.org/r/125069/#comment59345>

    KWindowConfig::saveWindowSize(dialog.windowHandle(), dialogConfig);


- Emmanuel Pescosta


On Sept. 26, 2015, 5:43 p.m., Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125069/
> -----------------------------------------------------------
> 
> (Updated Sept. 26, 2015, 5:43 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Repository: dolphin-plugins
> 
> 
> Description
> -------
> 
> Remove dependecy of kdelibs4support
> 
> 
> Diffs
> -----
> 
>   svn/fileviewsvnplugin.cpp faf6a0f 
>   svn/fileviewsvnplugin.h 2ba36bf 
>   svn/CMakeLists.txt 1627b92 
>   git/tagdialog.h 60563cd 
>   git/tagdialog.cpp 5da01ad 
>   git/pushdialog.cpp 1c82fa3 
>   git/pulldialog.cpp d51789e 
>   git/pushdialog.h 74da471 
>   git/pulldialog.h 861cb80 
>   git/fileviewgitplugin.h 3148b12 
>   git/fileviewgitplugin.cpp 7af37f0 
>   git/commitdialog.cpp 06222cf 
>   git/commitdialog.h dd5bfbf 
>   git/checkoutdialog.h 6d24c7d 
>   git/checkoutdialog.cpp 1eaecef 
>   git/CMakeLists.txt ea58c77 
>   CMakeLists.txt 76314bf 
> 
> Diff: https://git.reviewboard.kde.org/r/125069/diff/
> 
> 
> Testing
> -------
> 
> 1. KDialog to QDialog + QDialogButtonBox using convert-kdialog.pl
> 2. KLineEdit to QLineEdit
> 3. KTextEdit to QTextEdit
> 4. KVBox, KHBox to QWidget + QVBoxLayout, QWidget + QHBoxLayout using convert-kvbox.pl
> 
> 
> File Attachments
> ----------------
> 
> patch2
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/26/ec4f04f8-adc1-411c-a4ee-de43b0bec11c__patch2
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20150926/7a268316/attachment.htm>