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

Cedric Bosdonnat cedric.bosdonnat.ooo at free.fr
Wed May 25 01:03:39 PDT 2011

Hello Markus,

On Tue, 2011-05-24 at 21:35 +0200, Markus Mohrhard wrote:
> 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.

Sounds OK to me as the \brdr[tblr] are the first commands of a border
definition.

Many thanks for the fix.

--
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr