Review Request 120641: Turn virtuals into Q_DECL_OVERRIDE and rename a wrong override

Albert Astals Cid aacid at kde.org
Tue Oct 21 17:39:11 UTC 2014



> On oct. 18, 2014, 4:30 p.m., Aleix Pol Gonzalez wrote:
> > +1 Looks good to me.

Anyone has other comments? Can i has a shipit?


- Albert


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120641/#review68671
-----------------------------------------------------------


On oct. 18, 2014, 7:20 a.m., Albert Astals Cid wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120641/
> -----------------------------------------------------------
> 
> (Updated oct. 18, 2014, 7:20 a.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: attica
> 
> 
> Description
> -------
> 
> Turn virtuals that are not virtuals into Q_DECL_OVERRIDE
> 
> Also rename setNameAccessManager into setNam so that it actually overrides the base. I've serached in lxr.kde.org and there's no calls to any of the two so we're fine.
> 
> 
> Diffs
> -----
> 
>   src/deletejob.h 90a4578 
>   src/getjob.h 2ec9d76 
>   src/itemjob.h 8a373c3 
>   src/listjob.h 05f351e 
>   src/postjob.h 90bb8f4 
>   src/putjob.h 6267a78 
>   src/qtplatformdependent.cpp efd49aa 
>   src/qtplatformdependent_p.h 4be2f2e 
> 
> Diff: https://git.reviewboard.kde.org/r/120641/diff/
> 
> 
> Testing
> -------
> 
> Compiles
> 
> 
> Thanks,
> 
> Albert Astals Cid
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141021/d12f5480/attachment.html>


More information about the Kde-frameworks-devel mailing list