Hello Kendy, Cedric,<br><br>yes there might still be some problems, I only tried to fix that during pasting to calc all borders are drawn. I&#39;m not familiar with the RTF spec and the old code(before Cedric refactored it) was really ugly and even used some gotos.<br>
<br>This quick patch was more or less a result of <a href="https://bugs.freedesktop.org/show_bug.cgi?id=37429">https://bugs.freedesktop.org/show_bug.cgi?id=37429</a> as I noticed that borders were totally screwed up. If I can help in any way just mail me and I have a look at it.<br>
<br>Regards, Markus<br><br><div class="gmail_quote">2011/5/27 Cedric Bosdonnat <span dir="ltr">&lt;<a href="mailto:cedric.bosdonnat.ooo@free.fr">cedric.bosdonnat.ooo@free.fr</a>&gt;</span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Kendy,<br>
<div class="im"><br>
On Fri, 2011-05-27 at 07:43 +0200, Jan Holesovsky wrote:<br>
&gt; &gt; here is a short patch that sets all specified borders during parsing<br>
&gt; &gt; instead of only one border. The problem was that nBorderTyp was set<br>
&gt; &gt; for every border with the correct value but SetBorderLine was only<br>
&gt; &gt; called once after the do while loop. So now every time nBorderType<br>
&gt; &gt; will be overriden, I call SetBorderLine.<br>
&gt; &gt;<br>
&gt; &gt; If this patch is ok I think we should add it to the 3-4 branch.<br>
&gt;<br>
&gt; Cedric already approved the patch,<br>
<br>
</div>Well, I haven&#39;t actually pushed the patch (had to check as I wasn&#39;t<br>
sure). I was about to do it but I found some remaining problems with<br>
some border properties not being copied properly (width for example).<br>
<div class="im"><br>
&gt; but I am wondering - before, the<br>
&gt; nBorderTyp was set only when bTableDef was true; after your patch, it is<br>
&gt; set regardless of the bTableDef value. áIs that correct, or should that<br>
&gt; be in a block? áIf it is correct, can you please also change the<br>
&gt; indentation of the nBorderTyp = XYZ; part so that it does not look as if<br>
&gt; it is supposed to be part of the if ( bTableDef )?<br>
<br>
</div>I&#39;ll to have a look at that... I can&#39;t remember the reason of this, but<br>
it looks weird to me. IMHO setting it in all cases should be better...<br>
but I need to dive again into RTF specs and that parser code (that I<br>
partly rewrote some time ago)<br>
<div><div></div><div class="h5"><br>
--<br>
CÚdric Bosdonnat<br>
LibreOffice hacker<br>
<a href="http://documentfoundation.org" target="_blank">http://documentfoundation.org</a><br>
OOo Eclipse Integration developer<br>
<a href="http://cedric.bosdonnat.free.fr" target="_blank">http://cedric.bosdonnat.free.fr</a><br>
<br>
</div></div></blockquote></div><br>