<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 3e3b3ffcd3719da7ce7efe03a550799294b06531 by Inge Wallin to branch master.</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>