> On July 13, 2011, 12:52 p.m., Francesco Nwokeka wrote: > > config/behavior-config.h, line 36 > > <http://git.reviewboard.kde.org/r/101886/diff/2/?file=26781#file26781line36> > > > > QWodget *parent = 0 -> cooler. keep the * next to the var name
Source: http://techbase.kde.org/Policies/Kdelibs_Coding_Style#Whitespace > On July 13, 2011, 12:52 p.m., Francesco Nwokeka wrote: > > config/behavior-config.cpp, line 75 > > <http://git.reviewboard.kde.org/r/101886/diff/2/?file=26782#file26782line75> > > > > what's this for? if you don't have to output anything don't put the > > debug there. I find it useless @nwoki: FYI people do it because kDebug() will print out the method name at the start of the print, so it will output something. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101886/#review4679 ----------------------------------------------------------- On July 12, 2011, 3:02 p.m., Lasath Fernando wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101886/ > ----------------------------------------------------------- > > (Updated July 12, 2011, 3:02 p.m.) > > > Review request for Telepathy. > > > Summary > ------- > > This was rather a big refractor (for me at least), and this is the first time > I created any new classes. So I'm not sure I did quite a few things right. > > I created a new tab in the KCM, called Behaviour. I think we'll probably have > to change the labels on the radiobuttons to something that makes more sense. > > I also included "../app/telepathy-chat-ui.h" because I needed an enum from > it. I don't know if there's anything against doing that or not. > > I also am not sure I used a QButtonGroup correctly. > > Also, I noticed the other class didn't use a d-pointer, (even though a KCM is > supposed to be like a library file) so I didn't. > > There are many more things I was doubtful at the time, but I can't possibly > remember now (I shouldn't leave writing review requests till 2AM). Hopefully, > everything makes sense in the diff. > > And as always, my code is in my scratch repo: > http://quickgit.kde.org/?p=clones%2Ftelepathy-chat-handler%2Ffernando%2FdetachableTabs.git&a=shortlog&h=refs/heads/refractored_tabs > > > Diffs > ----- > > app/telepathy-chat-ui.h f3cab67 > app/telepathy-chat-ui.cpp fe0fb9b > config/CMakeLists.txt 48252a7 > config/appearance-config.h PRE-CREATION > config/appearance-config.cpp PRE-CREATION > config/behavior-config.h PRE-CREATION > config/behavior-config.cpp PRE-CREATION > config/behaviorconfig.ui PRE-CREATION > config/main-window.h e95bf93 > config/main-window.cpp 3a79da4 > config/main.cpp c47f493 > > Diff: http://git.reviewboard.kde.org/r/101886/diff > > > Testing > ------- > > Changed a (well, the only) setting, and the program changed behaviour as > expected. > > > Screenshots > ----------- > > > http://git.reviewboard.kde.org/r/101886/s/198/ > > > Thanks, > > Lasath > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
