> On Feb. 5, 2015, 3:32 p.m., Martin Klapetek wrote: > > src/personsmodel.cpp, line 37 > > <https://git.reviewboard.kde.org/r/122428/diff/1/?file=347013#file347013line37> > > > > I was always told that making private d-pointer a QObject is a very bad > > idea as it makes things unnecessarily heavy. Do we really need this as a > > QObject? If so, then it misses the Q_OBJECT macro too
Yes, well, I think it's better like this than with all the Q_PRIVATE_SLOT, because these too will exposte the forward declaration of AbstractContact.. That's why I did it like this. - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122428/#review75474 ----------------------------------------------------------- On Feb. 4, 2015, 5:55 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122428/ > ----------------------------------------------------------- > > (Updated Feb. 4, 2015, 5:55 p.m.) > > > Review request for KDEPIM and Telepathy. > > > Repository: libkpeople > > > Description > ------- > > Move the private methods into the p-impl. First, it's the correct thing to > do. Secondly, it was pulling an awkward dependency on abstractcontact.h from > the backends into a public header. > > > Diffs > ----- > > src/CMakeLists.txt d8b4875 > src/backends/abstractcontact.h 4d1edcd > src/personsmodel.h 239f4ed > src/personsmodel.cpp bc2bee6 > > Diff: https://git.reviewboard.kde.org/r/122428/diff/ > > > Testing > ------- > > Builds, tests pass, ktp-contactlist still works. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
