> On April 22, 2016, 6:19 a.m., Christian David wrote: > > Hi Mitch, > > > > according to the [Qt > > Docu](http://doc.qt.io/qt-5/qmetatype.html#qRegisterMetaType-1) > > ```qRegisterMetaType``` is (only) needed under some circumstances: > > > > To use the type T in QVariant, using Q_DECLARE_METATYPE() is > > sufficient. To use the type T in queued signal and slot connections, > > qRegisterMetaType<T>() must be called before the first connection is > > established. > > > > To prevent complicated issues if we use queued connections ourself in the > > future the ```qRegisterMetaType``` should go somewhere else. I think it > > should be ```mymoneymoney.cpp``` for ```MyMoneyMoney``` but I am unsure > > here.
In terms of logically where it ought to go I don't know, I'm not that familiar with the organization of the code base. However, in terms of satisfying the requirement that it "must be called before the first connection is established," making it a static variable initializer should be sufficient regardless of where it is placed. Static initializers are run when the .so is loaded so they'll be executed before any "real" code gets run. - Mitch ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127678/#review94753 ----------------------------------------------------------- On April 17, 2016, 11:03 p.m., Mitch Frazier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127678/ > ----------------------------------------------------------- > > (Updated April 17, 2016, 11:03 p.m.) > > > Review request for KMymoney. > > > Repository: kmymoney > > > Description > ------- > > Register metatypes to eliminate Qt Desinger warnings. > > Registering metatypes for a type used in a widget eliminates > the following types of warnings from Qt Designer on start up > (displayed when designer is started from a console window): > > QMetaProperty::read: Unable to handle unregistered datatype > '<class>' for property '<class>::<member>' > > > Diffs > ----- > > kmymoney/widgets/kmymoneycurrencyselector.cpp 41be539 > kmymoney/widgets/kmymoneyedit.cpp ac79db7 > > Diff: https://git.reviewboard.kde.org/r/127678/diff/ > > > Testing > ------- > > Tested dialogs that use the data types. > > > Thanks, > > Mitch Frazier > >
