Review Request 120640: Turn virtuals into Q_DECL_OVERRIDE and remove wrong override

Albert Astals Cid aacid at kde.org
Fri Oct 24 22:40:12 UTC 2014


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

(Updated Oct. 24, 2014, 10:40 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Repository: karchive


Description
-------

Turn virtuals into Q_DECL_OVERRIDE since it's what they really are.

Also remove putChar that tried to be an override but putChar is not virtual anymore in QIODevice and thus we don't need to reimplement it to say it's unsuported


Diffs
-----

  src/kcompressiondevice.h 39a3206 
  src/kgzipfilter.h d74ef8a 
  src/klimitediodevice_p.h dbc1ebf 
  src/knonefilter.h 9314ea2 
  src/kxzfilter.h 59f00f6 
  src/karchivedirectory.h 60225d0 
  src/karchivefile.h c7d2e0e 
  src/kbzip2filter.h 72e9756 

Diff: https://git.reviewboard.kde.org/r/120640/diff/


Testing
-------

Compiles


Thanks,

Albert Astals Cid

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


More information about the Kde-frameworks-devel mailing list