Re: Review Request 124344: Replace k3activelabel classes with QLabel in libkopete

> 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
[email protected]
https://mail.kde.org/mailman/listinfo/kopete-devel