----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127655/ -----------------------------------------------------------
(Updated May 4, 2016, 9:41 p.m.) Status ------ This change has been marked as submitted. Review request for KDE Frameworks, Alex Richardson and Michael Pyne. Changes ------- Submitted with commit a43fc021366eeaf630902827935d697c0d1f09b1 by Friedrich W. H. Kossebau to branch master. Repository: kcoreaddons Description ------- There is code in e.g. KXMLGUI which relies on KAboutData::applicationData(), without requiring the user to use KAboutData::setApplicationData(). So better be complete when initializing the data from the Q*Application metadata. Also - warn if there is no Q*App instance yet to set properties in KAboutData::setApplicationData - check and warn if KAboutData::applicationData is out-of-sync with qapp data - remove bogus check for empty display name, as the method defaults to componentname - unit tests KAboutDataTest::testSetOfQApplicationData/testPickupOfQApplicationData Diffs ----- autotests/CMakeLists.txt a7a6752 autotests/kaboutdataapplicationdatatest.cpp PRE-CREATION src/lib/kaboutdata.h 97c0f2b src/lib/kaboutdata.cpp ceb0c06 Diff: https://git.reviewboard.kde.org/r/127655/diff/ Testing ------- Added autotests pass. Thanks, Friedrich W. H. Kossebau
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
