<font color="#888888">Hello Bjoern,<br></font><br><div class="gmail_quote">2011/3/28 Bjoern Michaelsen <span dir="ltr">&lt;<a href="mailto:bjoern.michaelsen@canonical.com">bjoern.michaelsen@canonical.com</a>&gt;</span><br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Hi Theo,<br></blockquote><div> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
 The changes to<br>
   ChangeFontSizeImpl in editeng/source/editeng/editview.cxx<br>
however are assuming that aSet.Get( EE_CHAR_FONTHEIGHT_CJK ) returns a<br>
item with a which-id of EE_CHAR_FONTHEIGHT_CJK, which unfortunately we<br>
cannot assume currently (we need exactly the changes you are doing now<br>
everywhere, so that we finally can make that assumption). You would<br>
need to create a clone from the item you got and set the which-id<br>
explicitly before putting it in the second set, or leave the code as is<br>
for now.<br></blockquote><div><br>Hm. I&#39;m not sure -- this covers all callers in editeng, so wouldn&#39;t this be safe? Or is<br>this done by callers of editing as well?<br><br>Better leave them as is for now I think, until all code is converted.<br>
 </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
For the changes to<br>
   ConvertAndPutItems in editeng/source/editeng/editdoc2.cxx<br>
it seems to me that there is a &quot;delete pItem&quot; called now when<br>
eSourceUnit == eDestUnit which has not been there before.<br></blockquote><div class="im"><br>Correct, but we also call Clone() now in both cases, so I&#39;d <br>assume that necessary.<br>
<br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">In general, they are looking good, keep them coming!<br>
</blockquote><div> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">(*) Actually, the original is totally wrong here, and likely never<br>

worked as intended.<br><br></blockquote><div><br>Yep, noticed that as well.<br><br>Fixed patches coming up.<br><br>Kind regards,<br> - Theo<br> <br></div></div>