Review Request: Fix ObjectDescriptionModel<*>::staticMetaObject initialization on Qt5.
David Faure
faure at kde.org
Wed Oct 31 17:03:03 UTC 2012
> On Oct. 31, 2012, 4:16 p.m., Jon Severinsson wrote:
> > Over a week and still no comments. If no one objects in the next few days, I'm simply gonna push this without any "Ship It!".
I can't really review this code (without much research), but I completely support this "let's not let the fixes die for lack of review" strategy.
- David
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106976/#review21246
-----------------------------------------------------------
On Oct. 21, 2012, 11:43 a.m., Jon Severinsson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106976/
> -----------------------------------------------------------
>
> (Updated Oct. 21, 2012, 11:43 a.m.)
>
>
> Review request for KDE Frameworks and Phonon.
>
>
> Description
> -------
>
> The internals of QMetaObject has changed in Qt5, so we need two move
> the initialization into two different macros, one for Qt4 and one for Qt5.
> The generated C++ code will be the same after post processing on Qt4, so
> it shouldn't break anything, but it has only been compile tested on Qt5...
>
>
> Diffs
> -----
>
> phonon/objectdescriptionmodel.cpp fd903ae
>
> Diff: http://git.reviewboard.kde.org/r/106976/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Jon Severinsson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20121031/846bba1c/attachment.html>
More information about the Kde-frameworks-devel
mailing list