<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/115430/">https://git.reviewboard.kde.org/r/115430/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit f0f6d14367f85fb8e8c129e49801e05c318b8150 by Alex Merry to branch master.</pre>
<br />
<p>- Commit Hook</p>
<br />
<p>On February 3rd, 2014, 1:12 p.m. UTC, Alex Merry wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Frameworks and Alex Merry.</div>
<div>By Alex Merry.</div>
<p style="color: grey;"><i>Updated Feb. 3, 2014, 1:12 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kimageformats
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Add simple autotests
Two main tests: read the file and compare to a given image (loaded from
png), and write a file out and compare to a previously-written image.
The write test in particular is kind of fragile, but it should help
catch some errors where a code change was not intended to affect the
output of the plugin.
Tests are included for all the writable formats, as well as every
readable format that GIMP can write to (and so I could easily get an
example of).</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Tests build and pass.
Reference images PNG images are correct by inspection (checked with Gwenview on KDE 4).</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>autotests/write/rgb.eps <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgb.jp2 <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgb.pcx <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgb.pic <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgb.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgb.rgb <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgb.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgb.xv <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgba.pic <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgba.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgba.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/rgba.xv <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/writetest.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>tests/CMakeLists.txt <span style="color: grey">(b57287b94cc4c6f108737a3dc8831eecac57cf40)</span></li>
<li>autotests/read/xcf/simple-rgb-gimp-2.8.10.xcf <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xcf/simple-rgba-gimp-2.8.10.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xcf/simple-rgba-gimp-2.8.10.xcf <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xv/bw.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xv/bw.xv <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xv/rgb.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xv/rgb.xv <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/readtest.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bw.eps <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bw.jp2 <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bw.pcx <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bw.pic <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bw.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bw.rgb <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bw.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bw.xv <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bwa.pic <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bwa.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bwa.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/write/bwa.xv <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xcf/simple-bwa-gimp-2.8.10.xcf <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xcf/simple-rgb-gimp-2.8.10.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb-gimp-2.8.10-top-left.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgba.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgba.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xcf/simple-bw-gimp-2.8.10.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xcf/simple-bw-gimp-2.8.10.xcf <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/xcf/simple-bwa-gimp-2.8.10.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb-gimp-2.8.10-rle-top-left.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb-gimp-2.8.10-top-left.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/ras/rgb-gimp-2.8.10-standard.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/psd/rgb-gimp-2.8.10.psd <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/psd/rgb-gimp-2.8.10.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pic/rgba.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pic/rgba.pic <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pic/rgb.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pic/rgb.pic <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pic/bwa.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pic/bwa.pic <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pic/bw.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pic/bw.pic <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pcx/rgb.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pcx/rgb.pcx <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pcx/rgb-gimp-2.8.10.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pcx/rgb-gimp-2.8.10.pcx <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pcx/bw.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/pcx/bw.pcx <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/jp2/rgb.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/jp2/rgb.jp2 <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/jp2/bw.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/jp2/bw.jp2 <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/eps/rgb.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/eps/rgb.eps <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/eps/rgb-gimp-2.8.10.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/eps/rgb-gimp-2.8.10.eps <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/eps/bw.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/eps/bw.eps <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>CMakeLists.txt <span style="color: grey">(df7d02f379c5cd0e8040d4286701eb8cdc9f68ee)</span></li>
<li>autotests/read/rgb/rgb-gimp-2.8.10-uncompressed.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/rgb-gimp-2.8.10-rle.rgb <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/ras/rgb-gimp-2.8.10-standard.ras <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/bw.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/bw.rgb <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/rgb-gimp-2.8.10-aggressive-rle.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/rgb-gimp-2.8.10-aggressive-rle.rgb <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/rgb-gimp-2.8.10-rle.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/bwa.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/bw.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/bw.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/rgb.rgb <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/rgb.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/rgb/rgb-gimp-2.8.10-uncompressed.rgb <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb-gimp-2.8.10-bottom-left.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/bwa.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb-gimp-2.8.10-rle-top-left.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb-gimp-2.8.10-rle-bottom-left.tga <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb-gimp-2.8.10-rle-bottom-left.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/read/tga/rgb-gimp-2.8.10-bottom-left.tga <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/115430/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>