<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/107807/">http://git.reviewboard.kde.org/r/107807/</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 0d32fcd9b8e77a67b94ed2a82fc164f8ab9840e1 by Jean-Nicolas Artaud on behalf of Friedrich W. H. Kossebau to branch flake-configwidgets-boemann.</pre>
 <br />







<p>- Commit</p>


<br />
<p>On December 18th, 2012, 9:06 p.m., Friedrich W. H. Kossebau 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 Calligra.</div>
<div>By Friedrich W. H. Kossebau.</div>


<p style="color: grey;"><i>Updated Dec. 18, 2012, 9:06 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;">While I am pretty sure this patch does not break something, it's quite some lines, so I think you prefer to have another pair of eyes check this.

Patch removes usages of QString::null, resetting a QString by = QString() and unneeded reinitialisations in constructors.
</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;">No changes in unit tests.</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>filters/libmsooxml/MsooXmlCommonReaderDrawingMLImpl.h <span style="color: grey">(442ac67)</span></li>

 <li>filters/libmsooxml/MsooXmlDrawingTableStyleReader.cpp <span style="color: grey">(ec1963c)</span></li>

 <li>filters/libmsooxml/MsooXmlVmlReaderImpl.h <span style="color: grey">(0c3a7b1)</span></li>

 <li>filters/sheets/excel/sidewinder/excel.cpp <span style="color: grey">(72c3fbe)</span></li>

 <li>filters/sheets/excel/sidewinder/utils.cpp <span style="color: grey">(01b92a4)</span></li>

 <li>filters/sheets/xlsx/XlsxXmlWorksheetReader.cpp <span style="color: grey">(757d42b)</span></li>

 <li>filters/words/docx/DocxXmlDocumentReader.cpp <span style="color: grey">(634d3b9)</span></li>

 <li>filters/words/docx/DocxXmlNumberingReader.cpp <span style="color: grey">(6295a93)</span></li>

 <li>filters/words/msword-odf/texthandler.h <span style="color: grey">(e80f8ad)</span></li>

 <li>kexi/kexiutils/KexiCommandLinkButton.cpp <span style="color: grey">(35d1119)</span></li>

 <li>kexi/plugins/queries/kexiquerydesignerguieditor.cpp <span style="color: grey">(e3a9cab)</span></li>

 <li>krita/image/tests/kis_base_node_test.cpp <span style="color: grey">(2329b8f)</span></li>

 <li>krita/image/tests/kis_paint_device_test.cpp <span style="color: grey">(071e5e3)</span></li>

 <li>krita/plugins/formats/psd/psd_layer_section.cpp <span style="color: grey">(4e13395)</span></li>

 <li>krita/plugins/paintops/libbrush/abrbrush.cpp <span style="color: grey">(0deb375)</span></li>

 <li>krita/plugins/paintops/libbrush/kis_abr_brush.cpp <span style="color: grey">(3abf1dd)</span></li>

 <li>krita/plugins/paintops/libbrush/kis_abr_brush_collection.cpp <span style="color: grey">(1539a3a)</span></li>

 <li>krita/plugins/paintops/libbrush/kis_abr_translator.h <span style="color: grey">(bccc4bc)</span></li>

 <li>krita/plugins/paintops/libbrush/kis_abr_translator.cpp <span style="color: grey">(0367156)</span></li>

 <li>krita/ui/kra/kis_kra_loader.cpp <span style="color: grey">(8c8ecd9)</span></li>

 <li>libs/flake/KoShapeFactoryBase.h <span style="color: grey">(5722148)</span></li>

 <li>libs/flake/KoShapeSavingContext.h <span style="color: grey">(4801b29)</span></li>

 <li>libs/flake/KoToolManager.cpp <span style="color: grey">(4e09a8d)</span></li>

 <li>libs/koreport/wrtembed/KoReportDesigner.cpp <span style="color: grey">(e6eff07)</span></li>

 <li>libs/main/rdf/KoDocumentRdf.cpp <span style="color: grey">(4c60090)</span></li>

 <li>libs/odf/KoElementReference.cpp <span style="color: grey">(b51f97f)</span></li>

 <li>libs/odf/KoOdfBibliographyConfiguration.cpp <span style="color: grey">(19799cc)</span></li>

 <li>libs/odf/KoOdfLineNumberingConfiguration.cpp <span style="color: grey">(8fefb48)</span></li>

 <li>libs/odf/KoOdfNotesConfiguration.cpp <span style="color: grey">(a456eb0)</span></li>

 <li>libs/odf/KoOdfNumberDefinition.cpp <span style="color: grey">(d87120a)</span></li>

 <li>libs/odf/tests/kodomtest.cpp <span style="color: grey">(fd7defc)</span></li>

 <li>plugins/colorengines/lcms2/colorspaces/ycbcr_f32/YCbCrF32ColorSpace.h <span style="color: grey">(01f16ba)</span></li>

 <li>plugins/colorengines/lcms2/colorspaces/ycbcr_u16/YCbCrU16ColorSpace.h <span style="color: grey">(81796e5)</span></li>

 <li>plugins/colorengines/lcms2/colorspaces/ycbcr_u8/YCbCrU8ColorSpace.h <span style="color: grey">(78d00ca)</span></li>

 <li>stage/part/KPrViewModeSlidesSorter.cpp <span style="color: grey">(ca3d9b5)</span></li>

 <li>words/part/KWPageStyle.cpp <span style="color: grey">(0faab80)</span></li>

 <li>words/part/dialogs/KWAnchoringProperties.cpp <span style="color: grey">(558b48c)</span></li>

</ul>

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




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








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