> On July 24, 2012, 3:53 p.m., Sebastian Trueg wrote: > > libnepomukcore/resource/resourcemanager.cpp, line 91 > > <http://git.reviewboard.kde.org/r/105622/diff/1/?file=73688#file73688line91> > > > > legacy data is ignored here, isn't it? (Not very important though)
Yeah. I don't care that much about legacy data. Though I will write an app to convert all of it. - Vishesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105622/#review16305 ----------------------------------------------------------- On July 19, 2012, 2:35 p.m., Vishesh Handa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105622/ > ----------------------------------------------------------- > > Review request for Nepomuk and Sebastian Trueg. > > > Description > ------- > > Improve the concepts of kickoff lists and make the code a lot easier to > understand. Most of the "magic" of the kickoff lists has been removed to > make it more explicit, and easier to understand. > > Instead of having one kickOff list for everything. We now have 3 lists - > * One for initialized data. Maps uri -> ResourceData* > * One for nao:identifiers. Maps nao:identifier -> ResourceData* > * One for nie:urls. Maps nie:urls -> ResourceData* > > When constructing a ResourceData, the ResourceManager checks the kind > QUrl it has been given - a nepomukuri, a nie:url or nao:identifier. > Based on the results it passes the arguemnts to the ResourceData > constrcutor. > > This change was only possible cause with the DMS, we have made it clear > that all resource uri's will be of the form 'nepomuk:/'. This makes the > logic a lot simpler. > > > Diffs > ----- > > libnepomukcore/resource/resourcedata.h 1d0d761 > libnepomukcore/resource/resourcedata.cpp 37b48f2 > libnepomukcore/resource/resourcemanager.cpp 8f45289 > libnepomukcore/resource/resourcemanager_p.h 1547f5e > > Diff: http://git.reviewboard.kde.org/r/105622/diff/ > > > Testing > ------- > > I've run the tests, and there are no regressions, and this actually fixes on > test. > > But a lot of tests still fail (around 9), so there is still work to be done. > This is just the first step. > > > Thanks, > > Vishesh Handa > >
_______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
