Review Request 129170: Add KArchive::errorString() method to provide more details on KArchive errors

Romário Rios luizromario at gmail.com
Thu Oct 27 02:23:53 UTC 2016


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

(Updated Oct. 27, 2016, 2:23 a.m.)


Review request for KDE Frameworks.


Changes
-------

Applies fixes proposed by David. Still does not apply measures to fully internationalize framework as proposed by Friederich, but I wanted to get a review of my newest changes sooner and make sure they are good to go before internationalizing.


Repository: karchive


Description
-------

This method is similar to `QIODevice::errorString()`. I added a public `errorString()` method and a protected `setErrorString()` method, to allow `KArchive`'s subclasses to implement their own error messages. I also implemented most error messages from most subclasses.


Diffs (updated)
-----

  autotests/karchivetest.cpp d0fbf41 
  src/k7zip.h 5b95f49 
  src/k7zip.cpp 692b1db 
  src/kar.h 85bd650 
  src/kar.cpp 7204fb1 
  src/karchive.h b528a4a 
  src/karchive.cpp a1a160a 
  src/karchive_p.h 256620d 
  src/krcc.h 18c7d00 
  src/krcc.cpp 1947dd6 
  src/ktar.h 4bca898 
  src/ktar.cpp f70b155 
  src/kzip.h 84156c7 
  src/kzip.cpp 94d4276 

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


Testing
-------

I added `QVERIFY` calls after all errors in `karchivetests.cpp`. Perhaps we'll need more tests, but I'm not sure how to make an archive to fail in some specific way aside from the very basics ("file not found", etc.).


Thanks,

Romário Rios

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


More information about the Kde-frameworks-devel mailing list