> On Nov. 6, 2012, 3:58 a.m., Vishesh Handa wrote: > > src/core-impl/collections/nepomukcollection/NepomukCollection.cpp, line 117 > > <http://git.reviewboard.kde.org/r/107161/diff/1/?file=93131#file93131line117> > > > > In the Nepomuk code I generally prefer a 'using namespace > > Nepomuk::Vocabulary' at the top. > > > > This however is not Nepomuk code, so it's your choice :)
I am using Nepomuk::Vocabulary only once in the whole file. Thought it would be wasteful to use the namespace. - Phalgun ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107161/#review21472 ----------------------------------------------------------- On Nov. 9, 2012, 2:24 p.m., Phalgun Guduthur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107161/ > ----------------------------------------------------------- > > (Updated Nov. 9, 2012, 2:24 p.m.) > > > Review request for Amarok, Edward Hades Toroshchin and Vishesh Handa. > > > Description > ------- > > First footstep in completing the Nepomuk Collection. > > The non virtual methods belonging to TrackProvider implemented. > Repeated documentation to help vHanda understand the purpose of those > functions. > > > Diffs > ----- > > src/core-impl/collections/nepomukcollection/NepomukCollection.h d920ff6 > src/core-impl/collections/nepomukcollection/NepomukCollection.cpp 0593697 > > Diff: http://git.reviewboard.kde.org/r/107161/diff/ > > > Testing > ------- > > Minimal. > > > Thanks, > > Phalgun Guduthur > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
