Re: [Libreoffice] [Review][PUSHED3-4] set all borders correctly in SvxRTFParser::ReadBorderAttr instead of only one

Petr Mladek <pmladek -AT- suse.cz>
Mon, 30 May 2011 20:09:39 +0200

Cedric Bosdonnat píše v Pá 27. 05. 2011 v 16:03 +0200:

Hello Markus,

On Fri, 2011-05-27 at 14:02 +0200, Markus Mohrhard wrote:

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.

This quick patch was more or less a result of
https://bugs.freedesktop.org/show_bug.cgi?id=37429 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.


I pushed your patch


Updated subject to safe others time.


Best Regards,
Petr

Context

[Libreoffice] [Review] set all borders correctly in SvxRTFParser::ReadBorderAttr instead of only one · Markus Mohrhard
- Re: [Libreoffice] [Review] set all borders correctly in SvxRTFParser::ReadBorderAttr instead of only one · Cedric Bosdonnat
- Re: [Libreoffice] [Review] set all borders correctly in SvxRTFParser::ReadBorderAttr instead of only one · Jan Holesovsky
  - Re: [Libreoffice] [Review] set all borders correctly in SvxRTFParser::ReadBorderAttr instead of only one · Cedric Bosdonnat
    - Re: [Libreoffice] [Review] set all borders correctly in SvxRTFParser::ReadBorderAttr instead of only one · Markus Mohrhard
      - Re: [Libreoffice] [Review] set all borders correctly in SvxRTFParser::ReadBorderAttr instead of only one · Cedric Bosdonnat
        
        Re: [Libreoffice] [Review][PUSHED3-4] set all borders correctly in SvxRTFParser::ReadBorderAttr instead of only one · Petr Mladek

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.