Review Request 113881: Standardise d-pointer classes of kmediaplayer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113881/ --- Review request for KDE Frameworks. Repository: kdelibs Description --- Standardise d-pointer classes of kmediaplayer Use class Private instead of struct Data, for consistency with other KDE software, and move all private members to those classes. Diffs - staging/kmediaplayer/src/kmediaplayer/view.h 3bc849f5d93b5bb04c811c0c598d1f27fb25bfc2 staging/kmediaplayer/src/kmediaplayer/player.cpp a4618c4b1d93e2203cea496c100a0ed4ebe4d8dd staging/kmediaplayer/src/kmediaplayer/player.h c59c06fca0ea22994a7f00c6c509777be09370f1 staging/kmediaplayer/src/kmediaplayer/view.cpp eb2b779e8a4b044c1aa4f57b15567688da51a626 Diff: http://git.reviewboard.kde.org/r/113881/diff/ Testing --- Still builds. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 113881: Standardise d-pointer classes of kmediaplayer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113881/#review43726 --- Looks good to me. Actually I don't really understand what the Data pointer was before. It's only defined for one class but both have it as a member? It was used as a d-ptr only for some members? - Aleix Pol Gonzalez On Nov. 15, 2013, 12:20 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113881/ --- (Updated Nov. 15, 2013, 12:20 p.m.) Review request for KDE Frameworks. Repository: kdelibs Description --- Standardise d-pointer classes of kmediaplayer Use class Private instead of struct Data, for consistency with other KDE software, and move all private members to those classes. Diffs - staging/kmediaplayer/src/kmediaplayer/view.h 3bc849f5d93b5bb04c811c0c598d1f27fb25bfc2 staging/kmediaplayer/src/kmediaplayer/player.cpp a4618c4b1d93e2203cea496c100a0ed4ebe4d8dd staging/kmediaplayer/src/kmediaplayer/player.h c59c06fca0ea22994a7f00c6c509777be09370f1 staging/kmediaplayer/src/kmediaplayer/view.cpp eb2b779e8a4b044c1aa4f57b15567688da51a626 Diff: http://git.reviewboard.kde.org/r/113881/diff/ Testing --- Still builds. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 113881: Standardise d-pointer classes of kmediaplayer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113881/#review43758 --- Ship it! Ship It! - Kevin Ottens On Nov. 15, 2013, 12:20 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113881/ --- (Updated Nov. 15, 2013, 12:20 p.m.) Review request for KDE Frameworks. Repository: kdelibs Description --- Standardise d-pointer classes of kmediaplayer Use class Private instead of struct Data, for consistency with other KDE software, and move all private members to those classes. Diffs - staging/kmediaplayer/src/kmediaplayer/view.h 3bc849f5d93b5bb04c811c0c598d1f27fb25bfc2 staging/kmediaplayer/src/kmediaplayer/player.cpp a4618c4b1d93e2203cea496c100a0ed4ebe4d8dd staging/kmediaplayer/src/kmediaplayer/player.h c59c06fca0ea22994a7f00c6c509777be09370f1 staging/kmediaplayer/src/kmediaplayer/view.cpp eb2b779e8a4b044c1aa4f57b15567688da51a626 Diff: http://git.reviewboard.kde.org/r/113881/diff/ Testing --- Still builds. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 113881: Standardise d-pointer classes of kmediaplayer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113881/ --- (Updated Nov. 15, 2013, 9:03 p.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks. Repository: kdelibs Description --- Standardise d-pointer classes of kmediaplayer Use class Private instead of struct Data, for consistency with other KDE software, and move all private members to those classes. Diffs - staging/kmediaplayer/src/kmediaplayer/view.h 3bc849f5d93b5bb04c811c0c598d1f27fb25bfc2 staging/kmediaplayer/src/kmediaplayer/player.cpp a4618c4b1d93e2203cea496c100a0ed4ebe4d8dd staging/kmediaplayer/src/kmediaplayer/player.h c59c06fca0ea22994a7f00c6c509777be09370f1 staging/kmediaplayer/src/kmediaplayer/view.cpp eb2b779e8a4b044c1aa4f57b15567688da51a626 Diff: http://git.reviewboard.kde.org/r/113881/diff/ Testing --- Still builds. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 113881: Standardise d-pointer classes of kmediaplayer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113881/#review43776 --- This review has been submitted with commit 805ce7a14f5333509852b460e46e61b55d467490 by Alex Merry to branch frameworks. - Commit Hook On Nov. 15, 2013, 12:20 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113881/ --- (Updated Nov. 15, 2013, 12:20 p.m.) Review request for KDE Frameworks. Repository: kdelibs Description --- Standardise d-pointer classes of kmediaplayer Use class Private instead of struct Data, for consistency with other KDE software, and move all private members to those classes. Diffs - staging/kmediaplayer/src/kmediaplayer/view.h 3bc849f5d93b5bb04c811c0c598d1f27fb25bfc2 staging/kmediaplayer/src/kmediaplayer/player.cpp a4618c4b1d93e2203cea496c100a0ed4ebe4d8dd staging/kmediaplayer/src/kmediaplayer/player.h c59c06fca0ea22994a7f00c6c509777be09370f1 staging/kmediaplayer/src/kmediaplayer/view.cpp eb2b779e8a4b044c1aa4f57b15567688da51a626 Diff: http://git.reviewboard.kde.org/r/113881/diff/ Testing --- Still builds. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel