https://issues.apache.org/ooo/show_bug.cgi?id=119592
Andre <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |CONFIRMED CC| |[email protected] Ever confirmed|0 |1 --- Comment #4 from Andre <[email protected]> --- I am not really familiar with Writer code but it looks like m_nColumnCount is off by one by design, indicated by loops like for( sal_Int32 nCol = 0; nCol <= m_nColumnCount; ++nCol) ... With that in mind I understand the patch, but the original code looks like an error, ie the comparison if ( ... sal_Int32(m_aColDistance.size()) == m_nColumnCount) ...) looks like it should be if ( ... sal_Int32(m_aColDistance.size()) == m_nColumnCount+1) ...) which is basically what the patch does. My question is this: why not remove the sal_Int32(m_aColDistance.size()) == m_nColumnCount) part from the if-statement? -- You are receiving this mail because: You are the assignee for the bug.
