Review Request: Make q_ptr protected in TelepathyBaseJobPrivate and use it in subclasses

[Daniele E. Domenichelli]

> On 2010-05-26 11:58:28, Dario Freddi wrote:
> > /trunk/playground/network/telepathy-contactlist/add-contact-job.cpp, line 46
> > <http://reviewboard.kde.org/r/4153/diff/1/?file=27525#file27525line46>
> >
> >     initializing q_ptr should not be needed (please test though), as it is already initialized by the base class. So you're initializing it two times - removing this declaration should make it work fine.

yes indeed! That's why I removed it from every class...
I already tested it and it works.


- Daniele E.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4153/#review5873
-----------------------------------------------------------


On 2010-05-26 08:59:47, Daniele E. Domenichelli wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4153/
> -----------------------------------------------------------
> 
> (Updated 2010-05-26 08:59:47)
> 
> 
> Review request for telepathy and Dario Freddi.
> 
> 
> Summary
> -------
> 
> q_ptr is declared private in TelepathyBaseJobPrivate, but it should be protected.
> Derived classes should use the same q_ptr and not declare another one.
> 
> 
> Diffs
> -----
> 
>   /trunk/playground/network/telepathy-contactlist/add-contact-job.cpp 1130253 
>   /trunk/playground/network/telepathy-contactlist/add-contacts-to-group-job.cpp 1130253 
>   /trunk/playground/network/telepathy-contactlist/add-meta-contact-job.cpp 1130253 
>   /trunk/playground/network/telepathy-contactlist/remove-contacts-from-group-job.cpp 1130253 
>   /trunk/playground/network/telepathy-contactlist/remove-contacts-job.cpp 1130253 
>   /trunk/playground/network/telepathy-contactlist/telepathy-base-job_p.h 1130253 
> 
> Diff: http://reviewboard.kde.org/r/4153/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Daniele E.
> 
>