----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112472/#review39223 -----------------------------------------------------------
modemgsmcardinterface.cpp <http://git.reviewboard.kde.org/r/112472/#comment28921> You should avoid using QDBusReply and waitForFinished(). If operator id is a property updated by the propertiesChanged signal then you should cache its value and return the cached value here. modemgsmcardinterface.cpp <http://git.reviewboard.kde.org/r/112472/#comment28922> same here. - Lamarque Souza On Sept. 2, 2013, 9:54 p.m., Anant Kamath wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112472/ > ----------------------------------------------------------- > > (Updated Sept. 2, 2013, 9:54 p.m.) > > > Review request for Solid and Lamarque Souza. > > > Description > ------- > > Upgraded ModemGsmCardInterface to the MM 0.5 (atleast 0.5) spec. > > > Diffs > ----- > > dbus/introspection/mm-modem-gsm-card.xml e35821a > dbus/mm-modem-gsm-cardinterface.h 35f2dc4 > dbus/mm-modem-gsm-cardinterface.cpp 6bc2dc0 > modemgsmcardinterface.h 5f56a85 > modemgsmcardinterface.cpp 80d0a9c > > Diff: http://git.reviewboard.kde.org/r/112472/diff/ > > > Testing > ------- > > Builds, works. > > > Thanks, > > Anant Kamath > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
