Review Request 116568: Fixes to PIC image format handler

Alex Merry alex.merry at kde.org
Mon Mar 3 13:15:44 UTC 2014


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

(Updated March 3, 2014, 1:15 p.m.)


Review request for KDE Frameworks and Alex Merry.


Changes
-------

Transfer copyright/license information properly from merged files.


Repository: kimageformats


Description
-------

Fixes to PIC image format handler

Better error handling (returns false on error in read() and write()) and
use the correct format if there is no alpha channel.


Diffs (updated)
-----

  src/imageformats/CMakeLists.txt 44f07dd7bfb7daa1be21ececdfb5061a262e0fc8 
  src/imageformats/pic.cpp 9d8a7ede31c5c03a699d6a76c88aeb5e3d37ac4a 
  src/imageformats/pic_read.cpp 484c63426723e04e5c7e96ae5355ccceccab03f4 
  src/imageformats/pic_rw.h 2cc958927403de57049bbd7cb3200f0b7489da3c 
  src/imageformats/pic_write.cpp 0632eebd507e58e480856b53e71c24afc543de26 

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


Testing
-------

Builds and tests pass, both with and without https://git.reviewboard.kde.org/r/116567/ applied.


Thanks,

Alex Merry

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


More information about the Kde-frameworks-devel mailing list