[Merge] lp:~dandrader/qtubuntu/cursor into lp:qtubuntu

Daniel d'Andrada daniel.dandrada at canonical.com
Mon Nov 16 16:30:44 UTC 2015


On 13/11/2015 11:22, Lukáš Tinkl wrote:
> Review: Needs Fixing
>
> const char *qtCursorShapeToStr(Qt::CursorShape shape)
>
> You could use a QMetaObject and a QMetaEnum instead for those debug functions

I would need QMetaEnum::fromType(), which is available only in Qt 5.5. 
Don't know how to do it otherwise. QCursor is not a QObject. And even if 
QCursor derived from QObject, it would be quite a mess to have to dig 
through QMetaObject to get a QMetaEnum out of it. Ugh...

-- 
https://code.launchpad.net/~dandrader/qtubuntu/cursor/+merge/277285
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~dandrader/qtubuntu/cursor into lp:qtubuntu.



More information about the Ubuntu-reviews mailing list