> On Jan. 5, 2012, 2 p.m., Sebastian Trueg wrote: > > nepomuk/services/storage/resourcewatcherconnection.h, line 56 > > <http://git.reviewboard.kde.org/r/103522/diff/1/?file=44667#file44667line56> > > > > Why not lists? > > Vishesh Handa wrote: > I tried a variant list, but the signal wasn't getting emitted. Eventually > I got fed up and just used a variant. > > It can be changed later if someone can figure out how to send a variant > list.
Well, the DMS already does that in methods like setProperty or removeProperties. Just check the DataManagementAdaptor which uses QVariantList all over the place. - Sebastian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103522/#review9570 ----------------------------------------------------------- On Dec. 23, 2011, 4:12 p.m., Vishesh Handa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103522/ > ----------------------------------------------------------- > > (Updated Dec. 23, 2011, 4:12 p.m.) > > > Review request for Nepomuk, Martin Klapetek and Sebastian Trueg. > > > Description > ------- > > Implemented the propertyChanged signal in the Resource Watcher. > > DBus is retarded. It refuses to work if I sent the changed list as a > variant/variant list. It works if I send it as a string list though. > > > Diffs > ----- > > nepomuk/interfaces/org.kde.nepomuk.ResourceWatcherConnection.xml 8b175bc > nepomuk/services/storage/datamanagementmodel.h 2660837 > nepomuk/services/storage/datamanagementmodel.cpp b4e38aa > nepomuk/services/storage/lib/resourcewatcher.h ac4425d > nepomuk/services/storage/lib/resourcewatcher.cpp f6fc93c > nepomuk/services/storage/resourcewatcherconnection.h 6100fb4 > nepomuk/services/storage/resourcewatchermanager.h 707ddab > nepomuk/services/storage/resourcewatchermanager.cpp 6276254 > > Diff: http://git.reviewboard.kde.org/r/103522/diff/diff > > > Testing > ------- > > Works perfectly. Tried monitoring NAO:numericRating > > > Thanks, > > Vishesh Handa > >
_______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
