<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="http://git.reviewboard.kde.org/r/103070/">http://git.reviewboard.kde.org/r/103070/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">If you have tried it and it works, go ahead and commit it, can not find anything wrong in the patch.
Thanks!</pre>
<br />
<p>- Albert</p>
<br />
<p>On November 8th, 2011, 6:48 p.m., David Palacio wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Okular.</div>
<div>By David Palacio.</div>
<p style="color: grey;"><i>Updated Nov. 8, 2011, 6:48 p.m.</i></p>
<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;">Commit 39fc2a471e7cbbe4337ad095add6418b6bda256b introduced a change that only works with directories (i.e. not archives, zip, rar, etc.). It also makes a QImageReader test for each file twice, in extractImageFiles and pageSize. This change handles all cases (directories, zip, tar and rar) and does only one QImageReader test by merging extractImageFiles, pageSize and pages methods.</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;">Tested on master for Directories, Zip and Rar.</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>generators/comicbook/document.h <span style="color: grey">(1a1c72c)</span></li>
<li>generators/comicbook/document.cpp <span style="color: grey">(75c4334)</span></li>
<li>generators/comicbook/generator_comicbook.cpp <span style="color: grey">(d0b7b35)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103070/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>