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'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"><<a href="mailto:cedric.bosdonnat.ooo@free.fr">cedric.bosdonnat.ooo@free.fr</a>></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> > > here is a short patch that sets all specified borders during parsing<br> > > instead of only one border. The problem was that nBorderTyp was set<br> > > for every border with the correct value but SetBorderLine was only<br> > > called once after the do while loop. So now every time nBorderType<br> > > will be overriden, I call SetBorderLine.<br> > ><br> > > If this patch is ok I think we should add it to the 3-4 branch.<br> ><br> > Cedric already approved the patch,<br> <br> </div>Well, I haven't actually pushed the patch (had to check as I wasn'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> > but I am wondering - before, the<br> > nBorderTyp was set only when bTableDef was true; after your patch, it is<br> > set regardless of the bTableDef value. Is that correct, or should that<br> > be in a block? If it is correct, can you please also change the<br> > indentation of the nBorderTyp = XYZ; part so that it does not look as if<br> > it is supposed to be part of the if ( bTableDef )?<br> <br> </div>I'll to have a look at that... I can'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>