<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/113397/">http://git.reviewboard.kde.org/r/113397/</a>
     </td>
    </tr>
   </table>
   <br />



<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray solid;">
 <tr>
  <td>
   <h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been marked as submitted.</h1>
  </td>
 </tr>
</table>
<br />


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Gwenview and Aurélien Gâteau.</div>
<div>By Martin Kyral.</div>


<p style="color: grey;"><i>Updated Nov. 19, 2013, 9:22 a.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
gwenview
</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;">This is a review of patch enabling raw support in gwenview. It uses KDcraw to extract the embedded jpeg preview (most of the raw formats contain it for the purposes of quick viewing the photos on the camera display) or loads half-scaled preview (otherwise) so it is fast, implementing was quite easy and theoretically it enables gwenview to support everything dcraw supports (I haven't found a raw file that gwenview doesn't show with this patch). The patch does not perform full demosaicing nor any tweaks (set white balance or so) and I don't think it shall - digikam, darktable or rawtherapee are much more suited for developing the 'digital negatives'.

The patch fixes BZ#158788

The patch enables support for Nikon NEF and NRW formats, Canon's CR2 and CRW, Pentax PEF, Sony ARW, Fuji RAF, Olympus ORF, Minolta MRW, Panasonic RW2, Samsung SRW and Adobe DNG. If you want to try another format, just adding it's MIME type to the list in lib/mimetypeutils.cpp shall be enough.

The patch is feature-complete (generating proper thumbnails and viewing the raw images incl. EXIF metadata) and well usable.

There are some stuff to do (directory thumbnails and thumbnail in the file - properties dialog in particular), but these things are out of gwenview sources. I'll try to find out the proper place to place the fix (it seems that the candidates are kdelibs, kipi plugins or digikam).</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 some sample raw files taken from the camera reviews on http://www.photographyblog.com/ and bunch of my own photos. I have tried files of every single format to be supported (for DNG I tested on files taken by cameras from several vendors incl. Pentax, Leica and Nokia).</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>lib/document/loadingdocumentimpl.cpp <span style="color: grey">(fbad8ff)</span></li>

 <li>lib/CMakeLists.txt <span style="color: grey">(c6ffe14)</span></li>

 <li>CMakeLists.txt <span style="color: grey">(4dafb7e)</span></li>

 <li>lib/gwenviewconfig.kcfg <span style="color: grey">(231fd5e)</span></li>

 <li>lib/mimetypeutils.cpp <span style="color: grey">(546346f)</span></li>

 <li>lib/thumbnailprovider/thumbnailgenerator.cpp <span style="color: grey">(8b98904)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/113397/diff/" style="margin-left: 3em;">View Diff</a></p>







  </td>
 </tr>
</table>




  </div>
 </body>
</html>