<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/106942/">http://git.reviewboard.kde.org/r/106942/</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 59888763c1c3cca5bad0f56a93e64f79afea698f by Inge Wallin to branch kexi-altertable-staniek.</pre>
 <br />







<p>- Commit</p>


<br />
<p>On October 18th, 2012, 10:58 a.m., Inge Wallin 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 and C. Boemann.</div>
<div>By Inge Wallin.</div>


<p style="color: grey;"><i>Updated Oct. 18, 2012, 10:58 a.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;">It seems that the only differences between a master document and a normal ODT are:
 - An attribute text:global on the office:text tag (deprecated in ODF 1.2 but valid in 1.0 and 1.1)
 - A slightly different mime type

This patch adds support for roundtripping a master document even if it isn't handled correctly by Words yet.

Note that for LibreOffice to read it correctly, Friedrich's patch to KoDocument that was just put up for review is also needed.</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;">Testing with output from LO</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>words/part/KWDocument.h <span style="color: grey">(e0e4dd1)</span></li>

 <li>words/part/KWDocument.cpp <span style="color: grey">(6a6e7ae)</span></li>

 <li>words/part/KWOdfLoader.cpp <span style="color: grey">(215b332)</span></li>

 <li>words/part/KWOdfWriter.cpp <span style="color: grey">(6196dcc)</span></li>

 <li>words/part/words.desktop <span style="color: grey">(e734a18)</span></li>

 <li>words/part/wordspart.desktop <span style="color: grey">(c000807)</span></li>

</ul>

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




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








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