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

Markus Mohrhard <markus.mohrhard -AT- googlemail.com>
Tue, 24 May 2011 21:35:26 +0200

Hello,

here is a short patch that sets all specified borders during parsing instead
of only one border. The problem was that nBorderTyp was set for every border
with the correct value but SetBorderLine was only called once after the do
while loop. So now every time nBorderType will be overriden, I call
SetBorderLine.

If this patch is ok I think we should add it to the 3-4 branch.

Regards,
Markus

Attachment: 0001-set-all-border-lines-not-only-one.patch
Description: Binary data

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.