> On July 21, 2015, 10:33 p.m., Pali Rohár wrote: > > libkopete/ui/metacontactselectorwidget.cpp, line 21 > > <https://git.reviewboard.kde.org/r/124344/diff/2/?file=386363#file386363line21> > > > > Is this change needed? And if yes it is cosmetic, should be in separate > > commit and should change all includes qclass.h to QClass. Not just only > > one. Then we will have big mess. > > R.Harish Navnit wrote: > Since this whole patch does some porting to KDE4/Qt4, and deals directly > with the QLabel class, I thought it'd be sensible to do this change, keeping > in line with the Qt4 documentation. > > Revert it, nevertheless and create a separate patch ? > > Pali Rohár wrote: > Rewriting qclass.h to QClass is non functional change and should be done > in separate patch. And should change every qclass.h. > > R.Harish Navnit wrote: > Yes, I've reverted this one as well, in the updated patch : > https://git.reviewboard.kde.org/r/124344/diff/2-3/ > > Pali Rohár wrote: > In patch 3 I still see diff which change qlabel.h to QLabel.
Yeah, I noticed that too and I find it really strange. Revisions 1,2 - 3 show QLabel being reverted back to qlabel.h , but origin-3rd revision shows the exact opposite o.O - R.Harish ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124344/#review82775 ----------------------------------------------------------- On July 22, 2015, 1:05 a.m., R.Harish Navnit wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124344/ > ----------------------------------------------------------- > > (Updated July 22, 2015, 1:05 a.m.) > > > Review request for Kopete, Laurent Montel and Pali Rohár. > > > Repository: kopete > > > Description > ------- > > Use QLabel instead of K3ActiveLabel > > > Diffs > ----- > > libkopete/ui/addressbookselectorwidget.cpp > d1686ed855a94ee5a01db8a4ed0c04d573cd5be4 > libkopete/ui/addressbookselectorwidget_base.ui > de6ec4d24cd10eebfebc11b4fb9839be206552dc > libkopete/ui/metacontactselectorwidget.cpp > 6a45cf2ac9f6d904e7f7363d2fb7e7306f0c6c3a > libkopete/ui/metacontactselectorwidget_base.ui > 1f69a48c559ab3f002700068ec6a6c200a9242e3 > > Diff: https://git.reviewboard.kde.org/r/124344/diff/ > > > Testing > ------- > > Build succeeded. > > > Thanks, > > R.Harish Navnit > >
_______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel
