Re: Review Request 121078: Fix exports.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/#review70680 --- dolphin/src/CMakeLists.txt https://git.reviewboard.kde.org/r/121078/#comment49452 This one was missing from the commit and broke the build. I fixed it in 9cd55421bdadc663c947ddac9504a8d916e87012. - Luca Beltrame On Nov. 20, 2014, 3:29 a.m., Andrius da Costa Ribas wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/ --- (Updated Nov. 20, 2014, 3:29 a.m.) Review request for KDE Base Apps, KDE Frameworks and kdewin. Repository: kde-baseapps Description --- Use CMake's generate_export_header whenever possible, and fix some inconsistent exports. Also, use Q_DECL_EXPORT consistently across occurrences of kdemain for kdeinit apps (removing kdemacros.h where KDE_EXPORT was being in use). Diffs - dolphin/src/views/dolphinfileitemlistwidget.h 5f7c181 dolphin/src/views/dolphinitemlistview.h 7fd72e5 dolphin/src/views/versioncontrol/updateitemstatesthread.h 9b744d5 dolphin/src/views/versioncontrol/versioncontrolobserver.h 034ef8b dolphin/src/views/viewmodecontroller.h 9ea717a dolphin/src/views/viewproperties.h d3ea428 keditbookmarks/kbookmarkmodel/CMakeLists.txt 2f83af2 keditbookmarks/kbookmarkmodel/kbookmarkmodel_export.h b759bf9 keditbookmarks/main.cpp bc64193 konqueror/client/kfmclient.cpp 276452b konqueror/src/CMakeLists.txt 6831c88 konqueror/src/konq_statusbarmessagelabel.h a115d39 konqueror/src/konqmain.cpp 7a73231 konqueror/src/konqprivate_export.h 2dec78b dolphin/src/libdolphin_export.h ff0f125 dolphin/src/main.cpp 91d32a0 dolphin/src/kitemviews/private/kitemlistviewlayouter.h 0efcab1 dolphin/src/CMakeLists.txt c074908 dolphin/src/kitemviews/kfileitemlistview.h 9f07c9d dolphin/src/kitemviews/kfileitemlistwidget.h b7061d3 dolphin/src/kitemviews/kfileitemmodel.h 535e0f3 dolphin/src/kitemviews/kfileitemmodelrolesupdater.h e8b186a dolphin/src/kitemviews/kitemlistcontainer.h 55a9a11 dolphin/src/kitemviews/kitemlistcontroller.h 99df531 dolphin/src/kitemviews/kitemlistgroupheader.h 66fe929 dolphin/src/kitemviews/kitemlistheader.h fe2d0ac dolphin/src/kitemviews/kitemlistselectionmanager.h c4decd3 dolphin/src/kitemviews/kitemliststyleoption.h 782dd0e dolphin/src/kitemviews/kitemlistview.h cc0754f dolphin/src/kitemviews/kitemlistviewaccessible.h 674f1d1 dolphin/src/kitemviews/kitemlistwidget.h 07fc789 dolphin/src/kitemviews/kitemmodelbase.h 283cfa5 dolphin/src/kitemviews/kstandarditem.h 59190bf dolphin/src/kitemviews/kstandarditemlistgroupheader.h 871c93d dolphin/src/kitemviews/kstandarditemlistview.h eb07c7f dolphin/src/kitemviews/kstandarditemlistwidget.h 7683f84 dolphin/src/kitemviews/kstandarditemmodel.h 9d1db1a dolphin/src/kitemviews/private/kbaloorolesprovider.h f1ad5c7 dolphin/src/kitemviews/private/kfileitemmodeldirlister.h 94a0e39 dolphin/src/kitemviews/private/kfileitemmodelfilter.h e4fb0a2 dolphin/src/kitemviews/private/kitemlistheaderwidget.h b99f45f dolphin/src/kitemviews/private/kitemlistkeyboardsearchmanager.h 661178d dolphin/src/kitemviews/private/kitemlistrubberband.h aea58af dolphin/src/kitemviews/private/kitemlistselectiontoggle.h 8a1a857 dolphin/src/kitemviews/private/kitemlistsizehintresolver.h a0ad033 dolphin/src/kitemviews/private/kitemlistsmoothscroller.h ea56822 dolphin/src/kitemviews/private/kitemlistviewanimation.h e2e5338 dolphin/src/kitemviews/private/kpixmapmodifier.h 4f863c3 Diff: https://git.reviewboard.kde.org/r/121078/diff/ Testing --- It builds (MSVC2013 - 64bit) after this patch (along other patches I'm sending to review today). Kdebase-apps is still not very functional, though (missing icons and weird UI). Thanks, Andrius da Costa Ribas ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 121078: Fix exports.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/#review70681 --- Actually it's still broken: I didn't notice because I did not build tests: /srv/jenkins/workspace/kde-baseapps_frameworks_qt5/konqueror/src/konqapplication.h:28:46: error: expected initializer before ‘:’ token The line being class KONQ_TESTS_EXPORT KonquerorApplication : public KApplication - Luca Beltrame On Nov. 20, 2014, 3:29 a.m., Andrius da Costa Ribas wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/ --- (Updated Nov. 20, 2014, 3:29 a.m.) Review request for KDE Base Apps, KDE Frameworks and kdewin. Repository: kde-baseapps Description --- Use CMake's generate_export_header whenever possible, and fix some inconsistent exports. Also, use Q_DECL_EXPORT consistently across occurrences of kdemain for kdeinit apps (removing kdemacros.h where KDE_EXPORT was being in use). Diffs - dolphin/src/views/dolphinfileitemlistwidget.h 5f7c181 dolphin/src/views/dolphinitemlistview.h 7fd72e5 dolphin/src/views/versioncontrol/updateitemstatesthread.h 9b744d5 dolphin/src/views/versioncontrol/versioncontrolobserver.h 034ef8b dolphin/src/views/viewmodecontroller.h 9ea717a dolphin/src/views/viewproperties.h d3ea428 keditbookmarks/kbookmarkmodel/CMakeLists.txt 2f83af2 keditbookmarks/kbookmarkmodel/kbookmarkmodel_export.h b759bf9 keditbookmarks/main.cpp bc64193 konqueror/client/kfmclient.cpp 276452b konqueror/src/CMakeLists.txt 6831c88 konqueror/src/konq_statusbarmessagelabel.h a115d39 konqueror/src/konqmain.cpp 7a73231 konqueror/src/konqprivate_export.h 2dec78b dolphin/src/libdolphin_export.h ff0f125 dolphin/src/main.cpp 91d32a0 dolphin/src/kitemviews/private/kitemlistviewlayouter.h 0efcab1 dolphin/src/CMakeLists.txt c074908 dolphin/src/kitemviews/kfileitemlistview.h 9f07c9d dolphin/src/kitemviews/kfileitemlistwidget.h b7061d3 dolphin/src/kitemviews/kfileitemmodel.h 535e0f3 dolphin/src/kitemviews/kfileitemmodelrolesupdater.h e8b186a dolphin/src/kitemviews/kitemlistcontainer.h 55a9a11 dolphin/src/kitemviews/kitemlistcontroller.h 99df531 dolphin/src/kitemviews/kitemlistgroupheader.h 66fe929 dolphin/src/kitemviews/kitemlistheader.h fe2d0ac dolphin/src/kitemviews/kitemlistselectionmanager.h c4decd3 dolphin/src/kitemviews/kitemliststyleoption.h 782dd0e dolphin/src/kitemviews/kitemlistview.h cc0754f dolphin/src/kitemviews/kitemlistviewaccessible.h 674f1d1 dolphin/src/kitemviews/kitemlistwidget.h 07fc789 dolphin/src/kitemviews/kitemmodelbase.h 283cfa5 dolphin/src/kitemviews/kstandarditem.h 59190bf dolphin/src/kitemviews/kstandarditemlistgroupheader.h 871c93d dolphin/src/kitemviews/kstandarditemlistview.h eb07c7f dolphin/src/kitemviews/kstandarditemlistwidget.h 7683f84 dolphin/src/kitemviews/kstandarditemmodel.h 9d1db1a dolphin/src/kitemviews/private/kbaloorolesprovider.h f1ad5c7 dolphin/src/kitemviews/private/kfileitemmodeldirlister.h 94a0e39 dolphin/src/kitemviews/private/kfileitemmodelfilter.h e4fb0a2 dolphin/src/kitemviews/private/kitemlistheaderwidget.h b99f45f dolphin/src/kitemviews/private/kitemlistkeyboardsearchmanager.h 661178d dolphin/src/kitemviews/private/kitemlistrubberband.h aea58af dolphin/src/kitemviews/private/kitemlistselectiontoggle.h 8a1a857 dolphin/src/kitemviews/private/kitemlistsizehintresolver.h a0ad033 dolphin/src/kitemviews/private/kitemlistsmoothscroller.h ea56822 dolphin/src/kitemviews/private/kitemlistviewanimation.h e2e5338 dolphin/src/kitemviews/private/kpixmapmodifier.h 4f863c3 Diff: https://git.reviewboard.kde.org/r/121078/diff/ Testing --- It builds (MSVC2013 - 64bit) after this patch (along other patches I'm sending to review today). Kdebase-apps is still not very functional, though (missing icons and weird UI). Thanks, Andrius da Costa Ribas ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 121078: Fix exports.
On Nov. 20, 2014, 9 a.m., Luca Beltrame wrote: Actually it's still broken: I didn't notice because I did not build tests: /srv/jenkins/workspace/kde-baseapps_frameworks_qt5/konqueror/src/konqapplication.h:28:46: error: expected initializer before ‘:’ token The line being class KONQ_TESTS_EXPORT KonquerorApplication : public KApplication And finally fixed. - Luca --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/#review70681 --- On Nov. 20, 2014, 3:29 a.m., Andrius da Costa Ribas wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/ --- (Updated Nov. 20, 2014, 3:29 a.m.) Review request for KDE Base Apps, KDE Frameworks and kdewin. Repository: kde-baseapps Description --- Use CMake's generate_export_header whenever possible, and fix some inconsistent exports. Also, use Q_DECL_EXPORT consistently across occurrences of kdemain for kdeinit apps (removing kdemacros.h where KDE_EXPORT was being in use). Diffs - dolphin/src/views/dolphinfileitemlistwidget.h 5f7c181 dolphin/src/views/dolphinitemlistview.h 7fd72e5 dolphin/src/views/versioncontrol/updateitemstatesthread.h 9b744d5 dolphin/src/views/versioncontrol/versioncontrolobserver.h 034ef8b dolphin/src/views/viewmodecontroller.h 9ea717a dolphin/src/views/viewproperties.h d3ea428 keditbookmarks/kbookmarkmodel/CMakeLists.txt 2f83af2 keditbookmarks/kbookmarkmodel/kbookmarkmodel_export.h b759bf9 keditbookmarks/main.cpp bc64193 konqueror/client/kfmclient.cpp 276452b konqueror/src/CMakeLists.txt 6831c88 konqueror/src/konq_statusbarmessagelabel.h a115d39 konqueror/src/konqmain.cpp 7a73231 konqueror/src/konqprivate_export.h 2dec78b dolphin/src/libdolphin_export.h ff0f125 dolphin/src/main.cpp 91d32a0 dolphin/src/kitemviews/private/kitemlistviewlayouter.h 0efcab1 dolphin/src/CMakeLists.txt c074908 dolphin/src/kitemviews/kfileitemlistview.h 9f07c9d dolphin/src/kitemviews/kfileitemlistwidget.h b7061d3 dolphin/src/kitemviews/kfileitemmodel.h 535e0f3 dolphin/src/kitemviews/kfileitemmodelrolesupdater.h e8b186a dolphin/src/kitemviews/kitemlistcontainer.h 55a9a11 dolphin/src/kitemviews/kitemlistcontroller.h 99df531 dolphin/src/kitemviews/kitemlistgroupheader.h 66fe929 dolphin/src/kitemviews/kitemlistheader.h fe2d0ac dolphin/src/kitemviews/kitemlistselectionmanager.h c4decd3 dolphin/src/kitemviews/kitemliststyleoption.h 782dd0e dolphin/src/kitemviews/kitemlistview.h cc0754f dolphin/src/kitemviews/kitemlistviewaccessible.h 674f1d1 dolphin/src/kitemviews/kitemlistwidget.h 07fc789 dolphin/src/kitemviews/kitemmodelbase.h 283cfa5 dolphin/src/kitemviews/kstandarditem.h 59190bf dolphin/src/kitemviews/kstandarditemlistgroupheader.h 871c93d dolphin/src/kitemviews/kstandarditemlistview.h eb07c7f dolphin/src/kitemviews/kstandarditemlistwidget.h 7683f84 dolphin/src/kitemviews/kstandarditemmodel.h 9d1db1a dolphin/src/kitemviews/private/kbaloorolesprovider.h f1ad5c7 dolphin/src/kitemviews/private/kfileitemmodeldirlister.h 94a0e39 dolphin/src/kitemviews/private/kfileitemmodelfilter.h e4fb0a2 dolphin/src/kitemviews/private/kitemlistheaderwidget.h b99f45f dolphin/src/kitemviews/private/kitemlistkeyboardsearchmanager.h 661178d dolphin/src/kitemviews/private/kitemlistrubberband.h aea58af dolphin/src/kitemviews/private/kitemlistselectiontoggle.h 8a1a857 dolphin/src/kitemviews/private/kitemlistsizehintresolver.h a0ad033 dolphin/src/kitemviews/private/kitemlistsmoothscroller.h ea56822 dolphin/src/kitemviews/private/kitemlistviewanimation.h e2e5338 dolphin/src/kitemviews/private/kpixmapmodifier.h 4f863c3 Diff: https://git.reviewboard.kde.org/r/121078/diff/ Testing --- It builds (MSVC2013 - 64bit) after this patch (along other patches I'm sending to review today). Kdebase-apps is still not very functional, though (missing icons and weird UI). Thanks, Andrius da Costa Ribas ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 121078: Fix exports.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/ --- (Updated Nov. 20, 2014, 3:29 a.m.) Status -- This change has been marked as submitted. Review request for KDE Base Apps, KDE Frameworks and kdewin. Repository: kde-baseapps Description --- Use CMake's generate_export_header whenever possible, and fix some inconsistent exports. Also, use Q_DECL_EXPORT consistently across occurrences of kdemain for kdeinit apps (removing kdemacros.h where KDE_EXPORT was being in use). Diffs - dolphin/src/views/dolphinfileitemlistwidget.h 5f7c181 dolphin/src/views/dolphinitemlistview.h 7fd72e5 dolphin/src/views/versioncontrol/updateitemstatesthread.h 9b744d5 dolphin/src/views/versioncontrol/versioncontrolobserver.h 034ef8b dolphin/src/views/viewmodecontroller.h 9ea717a dolphin/src/views/viewproperties.h d3ea428 keditbookmarks/kbookmarkmodel/CMakeLists.txt 2f83af2 keditbookmarks/kbookmarkmodel/kbookmarkmodel_export.h b759bf9 keditbookmarks/main.cpp bc64193 konqueror/client/kfmclient.cpp 276452b konqueror/src/CMakeLists.txt 6831c88 konqueror/src/konq_statusbarmessagelabel.h a115d39 konqueror/src/konqmain.cpp 7a73231 konqueror/src/konqprivate_export.h 2dec78b dolphin/src/libdolphin_export.h ff0f125 dolphin/src/main.cpp 91d32a0 dolphin/src/kitemviews/private/kitemlistviewlayouter.h 0efcab1 dolphin/src/CMakeLists.txt c074908 dolphin/src/kitemviews/kfileitemlistview.h 9f07c9d dolphin/src/kitemviews/kfileitemlistwidget.h b7061d3 dolphin/src/kitemviews/kfileitemmodel.h 535e0f3 dolphin/src/kitemviews/kfileitemmodelrolesupdater.h e8b186a dolphin/src/kitemviews/kitemlistcontainer.h 55a9a11 dolphin/src/kitemviews/kitemlistcontroller.h 99df531 dolphin/src/kitemviews/kitemlistgroupheader.h 66fe929 dolphin/src/kitemviews/kitemlistheader.h fe2d0ac dolphin/src/kitemviews/kitemlistselectionmanager.h c4decd3 dolphin/src/kitemviews/kitemliststyleoption.h 782dd0e dolphin/src/kitemviews/kitemlistview.h cc0754f dolphin/src/kitemviews/kitemlistviewaccessible.h 674f1d1 dolphin/src/kitemviews/kitemlistwidget.h 07fc789 dolphin/src/kitemviews/kitemmodelbase.h 283cfa5 dolphin/src/kitemviews/kstandarditem.h 59190bf dolphin/src/kitemviews/kstandarditemlistgroupheader.h 871c93d dolphin/src/kitemviews/kstandarditemlistview.h eb07c7f dolphin/src/kitemviews/kstandarditemlistwidget.h 7683f84 dolphin/src/kitemviews/kstandarditemmodel.h 9d1db1a dolphin/src/kitemviews/private/kbaloorolesprovider.h f1ad5c7 dolphin/src/kitemviews/private/kfileitemmodeldirlister.h 94a0e39 dolphin/src/kitemviews/private/kfileitemmodelfilter.h e4fb0a2 dolphin/src/kitemviews/private/kitemlistheaderwidget.h b99f45f dolphin/src/kitemviews/private/kitemlistkeyboardsearchmanager.h 661178d dolphin/src/kitemviews/private/kitemlistrubberband.h aea58af dolphin/src/kitemviews/private/kitemlistselectiontoggle.h 8a1a857 dolphin/src/kitemviews/private/kitemlistsizehintresolver.h a0ad033 dolphin/src/kitemviews/private/kitemlistsmoothscroller.h ea56822 dolphin/src/kitemviews/private/kitemlistviewanimation.h e2e5338 dolphin/src/kitemviews/private/kpixmapmodifier.h 4f863c3 Diff: https://git.reviewboard.kde.org/r/121078/diff/ Testing --- It builds (MSVC2013 - 64bit) after this patch (along other patches I'm sending to review today). Kdebase-apps is still not very functional, though (missing icons and weird UI). Thanks, Andrius da Costa Ribas ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 121078: Fix exports.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/#review70678 --- This breaks compilation on Linux (was this tested?) and on CI: fatal error: libdolphin_export.h: No such file or directory - Luca Beltrame On Nov. 20, 2014, 3:29 a.m., Andrius da Costa Ribas wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/ --- (Updated Nov. 20, 2014, 3:29 a.m.) Review request for KDE Base Apps, KDE Frameworks and kdewin. Repository: kde-baseapps Description --- Use CMake's generate_export_header whenever possible, and fix some inconsistent exports. Also, use Q_DECL_EXPORT consistently across occurrences of kdemain for kdeinit apps (removing kdemacros.h where KDE_EXPORT was being in use). Diffs - dolphin/src/views/dolphinfileitemlistwidget.h 5f7c181 dolphin/src/views/dolphinitemlistview.h 7fd72e5 dolphin/src/views/versioncontrol/updateitemstatesthread.h 9b744d5 dolphin/src/views/versioncontrol/versioncontrolobserver.h 034ef8b dolphin/src/views/viewmodecontroller.h 9ea717a dolphin/src/views/viewproperties.h d3ea428 keditbookmarks/kbookmarkmodel/CMakeLists.txt 2f83af2 keditbookmarks/kbookmarkmodel/kbookmarkmodel_export.h b759bf9 keditbookmarks/main.cpp bc64193 konqueror/client/kfmclient.cpp 276452b konqueror/src/CMakeLists.txt 6831c88 konqueror/src/konq_statusbarmessagelabel.h a115d39 konqueror/src/konqmain.cpp 7a73231 konqueror/src/konqprivate_export.h 2dec78b dolphin/src/libdolphin_export.h ff0f125 dolphin/src/main.cpp 91d32a0 dolphin/src/kitemviews/private/kitemlistviewlayouter.h 0efcab1 dolphin/src/CMakeLists.txt c074908 dolphin/src/kitemviews/kfileitemlistview.h 9f07c9d dolphin/src/kitemviews/kfileitemlistwidget.h b7061d3 dolphin/src/kitemviews/kfileitemmodel.h 535e0f3 dolphin/src/kitemviews/kfileitemmodelrolesupdater.h e8b186a dolphin/src/kitemviews/kitemlistcontainer.h 55a9a11 dolphin/src/kitemviews/kitemlistcontroller.h 99df531 dolphin/src/kitemviews/kitemlistgroupheader.h 66fe929 dolphin/src/kitemviews/kitemlistheader.h fe2d0ac dolphin/src/kitemviews/kitemlistselectionmanager.h c4decd3 dolphin/src/kitemviews/kitemliststyleoption.h 782dd0e dolphin/src/kitemviews/kitemlistview.h cc0754f dolphin/src/kitemviews/kitemlistviewaccessible.h 674f1d1 dolphin/src/kitemviews/kitemlistwidget.h 07fc789 dolphin/src/kitemviews/kitemmodelbase.h 283cfa5 dolphin/src/kitemviews/kstandarditem.h 59190bf dolphin/src/kitemviews/kstandarditemlistgroupheader.h 871c93d dolphin/src/kitemviews/kstandarditemlistview.h eb07c7f dolphin/src/kitemviews/kstandarditemlistwidget.h 7683f84 dolphin/src/kitemviews/kstandarditemmodel.h 9d1db1a dolphin/src/kitemviews/private/kbaloorolesprovider.h f1ad5c7 dolphin/src/kitemviews/private/kfileitemmodeldirlister.h 94a0e39 dolphin/src/kitemviews/private/kfileitemmodelfilter.h e4fb0a2 dolphin/src/kitemviews/private/kitemlistheaderwidget.h b99f45f dolphin/src/kitemviews/private/kitemlistkeyboardsearchmanager.h 661178d dolphin/src/kitemviews/private/kitemlistrubberband.h aea58af dolphin/src/kitemviews/private/kitemlistselectiontoggle.h 8a1a857 dolphin/src/kitemviews/private/kitemlistsizehintresolver.h a0ad033 dolphin/src/kitemviews/private/kitemlistsmoothscroller.h ea56822 dolphin/src/kitemviews/private/kitemlistviewanimation.h e2e5338 dolphin/src/kitemviews/private/kpixmapmodifier.h 4f863c3 Diff: https://git.reviewboard.kde.org/r/121078/diff/ Testing --- It builds (MSVC2013 - 64bit) after this patch (along other patches I'm sending to review today). Kdebase-apps is still not very functional, though (missing icons and weird UI). Thanks, Andrius da Costa Ribas ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 121078: Fix exports.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/#review70148 --- Ship it! Looks good to me, really useful and definitely a step forward towards a ported dolphin. I'm saying shipit because I've ended having to do that in many projects already. Thanks! - Aleix Pol Gonzalez On Nov. 8, 2014, 10:25 p.m., Andrius da Costa Ribas wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/ --- (Updated Nov. 8, 2014, 10:25 p.m.) Review request for KDE Base Apps, KDE Frameworks and kdewin. Repository: kde-baseapps Description --- Use CMake's generate_export_header whenever possible, and fix some inconsistent exports. Also, use Q_DECL_EXPORT consistently across occurrences of kdemain for kdeinit apps (removing kdemacros.h where KDE_EXPORT was being in use). Diffs - dolphin/src/views/dolphinfileitemlistwidget.h 5f7c181 dolphin/src/views/dolphinitemlistview.h 7fd72e5 dolphin/src/views/versioncontrol/updateitemstatesthread.h 9b744d5 dolphin/src/views/versioncontrol/versioncontrolobserver.h 034ef8b dolphin/src/views/viewmodecontroller.h 9ea717a dolphin/src/views/viewproperties.h d3ea428 keditbookmarks/kbookmarkmodel/CMakeLists.txt 2f83af2 keditbookmarks/kbookmarkmodel/kbookmarkmodel_export.h b759bf9 keditbookmarks/main.cpp bc64193 konqueror/client/kfmclient.cpp 276452b konqueror/src/CMakeLists.txt 6831c88 konqueror/src/konq_statusbarmessagelabel.h a115d39 konqueror/src/konqmain.cpp 7a73231 konqueror/src/konqprivate_export.h 2dec78b dolphin/src/libdolphin_export.h ff0f125 dolphin/src/main.cpp 91d32a0 dolphin/src/kitemviews/private/kitemlistviewlayouter.h 0efcab1 dolphin/src/CMakeLists.txt c074908 dolphin/src/kitemviews/kfileitemlistview.h 9f07c9d dolphin/src/kitemviews/kfileitemlistwidget.h b7061d3 dolphin/src/kitemviews/kfileitemmodel.h 535e0f3 dolphin/src/kitemviews/kfileitemmodelrolesupdater.h e8b186a dolphin/src/kitemviews/kitemlistcontainer.h 55a9a11 dolphin/src/kitemviews/kitemlistcontroller.h 99df531 dolphin/src/kitemviews/kitemlistgroupheader.h 66fe929 dolphin/src/kitemviews/kitemlistheader.h fe2d0ac dolphin/src/kitemviews/kitemlistselectionmanager.h c4decd3 dolphin/src/kitemviews/kitemliststyleoption.h 782dd0e dolphin/src/kitemviews/kitemlistview.h cc0754f dolphin/src/kitemviews/kitemlistviewaccessible.h 674f1d1 dolphin/src/kitemviews/kitemlistwidget.h 07fc789 dolphin/src/kitemviews/kitemmodelbase.h 283cfa5 dolphin/src/kitemviews/kstandarditem.h 59190bf dolphin/src/kitemviews/kstandarditemlistgroupheader.h 871c93d dolphin/src/kitemviews/kstandarditemlistview.h eb07c7f dolphin/src/kitemviews/kstandarditemlistwidget.h 7683f84 dolphin/src/kitemviews/kstandarditemmodel.h 9d1db1a dolphin/src/kitemviews/private/kbaloorolesprovider.h f1ad5c7 dolphin/src/kitemviews/private/kfileitemmodeldirlister.h 94a0e39 dolphin/src/kitemviews/private/kfileitemmodelfilter.h e4fb0a2 dolphin/src/kitemviews/private/kitemlistheaderwidget.h b99f45f dolphin/src/kitemviews/private/kitemlistkeyboardsearchmanager.h 661178d dolphin/src/kitemviews/private/kitemlistrubberband.h aea58af dolphin/src/kitemviews/private/kitemlistselectiontoggle.h 8a1a857 dolphin/src/kitemviews/private/kitemlistsizehintresolver.h a0ad033 dolphin/src/kitemviews/private/kitemlistsmoothscroller.h ea56822 dolphin/src/kitemviews/private/kitemlistviewanimation.h e2e5338 dolphin/src/kitemviews/private/kpixmapmodifier.h 4f863c3 Diff: https://git.reviewboard.kde.org/r/121078/diff/ Testing --- It builds (MSVC2013 - 64bit) after this patch (along other patches I'm sending to review today). Kdebase-apps is still not very functional, though (missing icons and weird UI). Thanks, Andrius da Costa Ribas ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 121078: Fix exports.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121078/ --- Review request for KDE Base Apps, KDE Frameworks and kdewin. Repository: kde-baseapps Description --- Use CMake's generate_export_header whenever possible, and fix some inconsistent exports. Also, use Q_DECL_EXPORT consistently across occurrences of kdemain for kdeinit apps (removing kdemacros.h where KDE_EXPORT was being in use). Diffs - dolphin/src/views/dolphinfileitemlistwidget.h 5f7c181 dolphin/src/views/dolphinitemlistview.h 7fd72e5 dolphin/src/views/versioncontrol/updateitemstatesthread.h 9b744d5 dolphin/src/views/versioncontrol/versioncontrolobserver.h 034ef8b dolphin/src/views/viewmodecontroller.h 9ea717a dolphin/src/views/viewproperties.h d3ea428 keditbookmarks/kbookmarkmodel/CMakeLists.txt 2f83af2 keditbookmarks/kbookmarkmodel/kbookmarkmodel_export.h b759bf9 keditbookmarks/main.cpp bc64193 konqueror/client/kfmclient.cpp 276452b konqueror/src/CMakeLists.txt 6831c88 konqueror/src/konq_statusbarmessagelabel.h a115d39 konqueror/src/konqmain.cpp 7a73231 konqueror/src/konqprivate_export.h 2dec78b dolphin/src/libdolphin_export.h ff0f125 dolphin/src/main.cpp 91d32a0 dolphin/src/kitemviews/private/kitemlistviewlayouter.h 0efcab1 dolphin/src/CMakeLists.txt c074908 dolphin/src/kitemviews/kfileitemlistview.h 9f07c9d dolphin/src/kitemviews/kfileitemlistwidget.h b7061d3 dolphin/src/kitemviews/kfileitemmodel.h 535e0f3 dolphin/src/kitemviews/kfileitemmodelrolesupdater.h e8b186a dolphin/src/kitemviews/kitemlistcontainer.h 55a9a11 dolphin/src/kitemviews/kitemlistcontroller.h 99df531 dolphin/src/kitemviews/kitemlistgroupheader.h 66fe929 dolphin/src/kitemviews/kitemlistheader.h fe2d0ac dolphin/src/kitemviews/kitemlistselectionmanager.h c4decd3 dolphin/src/kitemviews/kitemliststyleoption.h 782dd0e dolphin/src/kitemviews/kitemlistview.h cc0754f dolphin/src/kitemviews/kitemlistviewaccessible.h 674f1d1 dolphin/src/kitemviews/kitemlistwidget.h 07fc789 dolphin/src/kitemviews/kitemmodelbase.h 283cfa5 dolphin/src/kitemviews/kstandarditem.h 59190bf dolphin/src/kitemviews/kstandarditemlistgroupheader.h 871c93d dolphin/src/kitemviews/kstandarditemlistview.h eb07c7f dolphin/src/kitemviews/kstandarditemlistwidget.h 7683f84 dolphin/src/kitemviews/kstandarditemmodel.h 9d1db1a dolphin/src/kitemviews/private/kbaloorolesprovider.h f1ad5c7 dolphin/src/kitemviews/private/kfileitemmodeldirlister.h 94a0e39 dolphin/src/kitemviews/private/kfileitemmodelfilter.h e4fb0a2 dolphin/src/kitemviews/private/kitemlistheaderwidget.h b99f45f dolphin/src/kitemviews/private/kitemlistkeyboardsearchmanager.h 661178d dolphin/src/kitemviews/private/kitemlistrubberband.h aea58af dolphin/src/kitemviews/private/kitemlistselectiontoggle.h 8a1a857 dolphin/src/kitemviews/private/kitemlistsizehintresolver.h a0ad033 dolphin/src/kitemviews/private/kitemlistsmoothscroller.h ea56822 dolphin/src/kitemviews/private/kitemlistviewanimation.h e2e5338 dolphin/src/kitemviews/private/kpixmapmodifier.h 4f863c3 Diff: https://git.reviewboard.kde.org/r/121078/diff/ Testing --- It builds (MSVC2013 - 64bit) after this patch (along other patches I'm sending to review today). Kdebase-apps is still not very functional, though (missing icons and weird UI). Thanks, Andrius da Costa Ribas ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
